| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
|
| |
Master build fails with: lib/common/Timer.cpp:171:10: error: reference cannot
be bound to dereferenced null pointer in well-defined C++ code; pointer may be
assumed to always convert to true [-Werror,-Wundefined-bool-conversion].
Cannot get regtests to pass on this feature branch without fixing this.
Also remove unused code from Timers.
|
|
|
|
| |
Thanks to Reinhard Tartler for pointing out the problem and suggesting the fix!
|
|
|
|
|
|
|
| |
Should hopefully reduce random failures of timing-dependent tests which
rely on subsecond precision for accuracy.
(cherry picked from commit c845b2e39fffeb560983a301d810616a6495469d)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Ideally timers would be perfectly accurate and we could sleep for 1.0 seconds,
but on OSX in particular they could fire 50-100 ms late (I've seen 4 ms in
practice) and we don't want the tests to fail because of this, because we don't
really need that kind of precision in practice. So we reduce the timer
intervals by 100ms to be safe.
Increase logging level and add timestamps in testcommon. Should help to debug
frequent test failures such as Condition [t2.HasExpired()] on OSX hosts on
Travis.
Add a SettingsGuard to the Console logger for its specific settings, such as
showing times and microseconds.
(cherry picked from commit a5e6cff1d435329b0121417ed9509e315ce0edd5)
(cherry picked from commit 8d02eebce553ed822e0fcd60d6e319384e15ba4b)
|
| |
|
|
|
|
|
| |
64-bit Solaris has _FILE_OFFSET_BITS defined (to 64), which was wrongly causing
the substitute lseek in lib/intercept to enter the 32-bit branch.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Add platform detection that's compatible with the old buildsystem and its m4
autoconfigury.
Make Travis build and test using CMake as a separate target.
Remove unused function BoxGetTemporaryDirectoryName() to fixes compile using
CMake on Unix.
|
|
|
|
|
|
|
| |
Rewrite some macros in forms that can be parsed by our parser written in CMake
language (fake m4 with regexps).
Fix incorrect conditional includes and incorrect #if/#ifdef assumptions.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
We need to use CreateProcessW() instead of system() to pass these filenames
to the rd command.
Fixes every test that runs after test_unicode_filenames_can_be_backed_up
in test/bbackupd.
|
| |
|
| |
|
|
|
|
|
| |
Returning an uninitialised value was causing copying of random amounts of data
in bbackupd on command socket disconnect, and crashes.
|
|
|
|
| |
Allows us to remove cleanup code, because it happens automatically.
|
|
|
|
|
| |
Like HousekeepStoreAccount and BackupStoreCheck, throwing an exception during
the destructor will terminate the application in an "unusual way".
|
| |
|
|
|
|
| |
Use INVALID_HANDLE_VALUE for invalid handle values, instead of 0.
|
|
|
|
| |
Windows.
|
|
|
|
|
| |
GetErrorMessage() already includes the error number in the text string that
it returns, so we don't need to include it again.
|
|
|
|
|
| |
Like HousekeepStoreAccount, throwing an exception during the destructor will
terminate the application in an "unusual way".
|
| |
|
|
|
|
|
| |
We should always use closesocket() instead, which the Close() method already
does, so just call it instead.
|
|
|
|
|
|
| |
Destructors aren't supposed to throw exceptions, and they can be called
while cleaning up from a previous exception, which will terminate the
application, so just log the error and carry on.
|
|
|
|
|
| |
I think something is on the PATH in Appveyor that makes rmdir /s/q fail,
especially when run through Perl.
|
|
|
|
| |
Break dependendency of test/bbackupd on individual files from other modules.
|
| |
|
|
|
|
| |
Use the rmdir command instead of rm -rf.
|
| |
|
|
|
|
| |
debugging possible
|
| |
|
|\ |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
Allows tests to depend on lib/bbackupd instead of bin/bbackupd, which was
always a hack, and really doesn't work with CMake.
|
| |
| |
| |
| |
| |
| |
| | |
These functions check whether the beginning and end of a std::string match
a supplied prefix or suffix, and remove that prefix or suffix, returning the
remaining part. They are almost but not entirely trivial, and giving them
names makes the code that uses them much more readable.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
These functions check whether the beginning and end of a std::string match
a supplied prefix or suffix, and remove that prefix or suffix, returning the
remaining part. They are almost but not entirely trivial, and giving them
names makes the code that uses them much more readable.
|
|\ \
| | |
| | |
| | | |
Some local commits were not pushed before, sorry.
|
| | | |
|
| |/ |
|
|\| |
|