summaryrefslogtreecommitdiffstats
path: root/onlineupdate
Commit message (Collapse)AuthorAgeFilesLines
* Cut down on -pthread/-lpthread proliferationStephan Bergmann2019-04-252-2/+0
| | | | | | | | | | | | | | | | | | | | | Building against libstdc++ effectively always requires -pthread anyway (as various standard C++ headers require it, see the comment added to solenv/gbuild/platform/unxgcc.mk), so many explicit uses of -pthread/-lpthread can be removed. Doing a (partial) test build on Linux with Clang -stdlib=libc++ suggests that libc++ indeed doesn't need -pthread as libstdc++ does. The remaining uses of -pthread/-lpthread are mostly in configure.ac for the various BSDs (which somebody else might want to clean up now), and related to external projects. I tried to be careful to remove -pthread/-lpthread from makefiles only when C++ object files are involved (so -pthread will now be included on the link command line by default). Change-Id: I936e082839cb9a434bd273ce5a1f187a4245dfa1 Reviewed-on: https://gerrit.libreoffice.org/71291 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Drop UNICODE/_UNICODE definesMike Kaganski2019-04-048-24/+1
| | | | | | | | | | | | The code is using expicit (mostly W) Windows API, and is independent from the macro. Removing it here allows to catch places where some UNICODE-dependent macro is used unintentionally. Change-Id: I5dff40aecfc3c3dc7fc4cf7271a995a675943a45 Reviewed-on: https://gerrit.libreoffice.org/70237 Reviewed-by: Michael Stahl <Michael.Stahl@cib.de> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com> Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
* work around Werror=format-truncation in onlineupdate for nowMarkus Mohrhard2019-02-151-5/+10
| | | | | | | Change-Id: I6230cc8c76313b5d180ef3fbebec4bc20b794150 Reviewed-on: https://gerrit.libreoffice.org/67879 Tested-by: Jenkins Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Rename Mac OS X to official name macOS in comments and documentationBartosz Kosiorek2018-11-291-3/+3
| | | | | | | Change-Id: I651b7f202fa52ff5f5357a11aa72c43eb7dc7f95 Reviewed-on: https://gerrit.libreoffice.org/64102 Tested-by: Jenkins Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* fix off by one errorMarkus Mohrhard2018-11-181-1/+1
| | | | | | | | | BLRG-PT-18-004 Change-Id: I9cd2582aebf2b33d40f23254047c35c8dee90f4b Reviewed-on: https://gerrit.libreoffice.org/63508 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Rename online updater functions and strcmp relpathJan-Marek Glogowski2018-09-101-32/+33
| | | | | | | | | | | | | | | | Try to be more explicit about the newly allocated absolute path and the relative offset and actually compare the relative part. -get_full_path(const NS_tchar *relpath) +new_absolute_path(const NS_tchar *relpath) -get_relative_path(const NS_tchar *fullpath) +get_relative_offset(const NS_tchar *abs_path) Change-Id: I02abd722fc89a1688502b9c5cd375954f9f0bc05 Reviewed-on: https://gerrit.libreoffice.org/60168 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
* Fix some std::unique_ptr array allocsJan-Marek Glogowski2018-09-071-16/+16
| | | | | | | | | | | | Found by reviewing the output of and the code around git grep -n "unique_ptr.*new.*\[.*\]" | grep -v "\[\]" The onlineupdater code needs a little bit more attention. Change-Id: I8b70c7da7db60af52bfac12314a21602ede8bfc0 Reviewed-on: https://gerrit.libreoffice.org/60162 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* updater: Tone down the geekiness in user-facing stringsAdolfo Jayme Barrientos2018-03-202-8/+8
| | | | | | Pro tip: The user doesn’t have to know that the updater is a separate program. Change-Id: I04c447a16a322b7f4139e69f6cababc797148bb1
* Blind fixStephan Bergmann2018-01-171-1/+1
| | | | | | | ...in --enable-online-update=mar code, after 7d8e94444d989d0ac4a4055b207726708e9ec0da "convert a<b?a:b to std::min(a,b)" Change-Id: I8d03b2fc785595267c855c8ac9953d35782c7718
* Fix typosAndrea Gelmini2018-01-162-2/+2
| | | | | | | Change-Id: Ia976371bf52eb1216d8abe728d80bbb87a3c38a2 Reviewed-on: https://gerrit.libreoffice.org/47858 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* Missing includeStephan Bergmann2018-01-161-0/+1
| | | | Change-Id: Id5c7ff8b9d342b557bc87b1ac6f3e2fbc8437d69
* convert a<b?a:b to std::min(a,b)Noel Grandin2018-01-151-1/+1
| | | | | | | | | | | | with something like git grep -nP '(.*)\s*<\s*(.*)\s*\?\s*\g1\s*:\s*\g2' -- *.?xx Change-Id: Id5078b35961847feb78a66204fdb7598ee63fd23 Note: we also convert a>b?b:a Reviewed-on: https://gerrit.libreoffice.org/47736 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Updater Icon: Shrink svg fileandreas kainz2017-12-302-649/+1
| | | | | | | Change-Id: I8a45d4e620b1865dda28eb56a89207226eeac9b2 Reviewed-on: https://gerrit.libreoffice.org/47202 Reviewed-by: andreas_kainz <kainz.a@gmail.com> Tested-by: andreas_kainz <kainz.a@gmail.com>
* Fix typosAndrea Gelmini2017-11-191-1/+1
| | | | | | | Change-Id: I964528cf4f3b81230c683011e1878b28378b0014 Reviewed-on: https://gerrit.libreoffice.org/44915 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* chmod 0755 -> 0644 for some source filesStephan Bergmann2017-10-231-0/+0
| | | | Change-Id: Ibe5b5e03374419c2c23cd6559ab213d2dc2fcc66
* onlineupdate: work around GCC7 -Werror=implicit-fallthroughMichael Stahl2017-09-132-1/+2
| | | | | | This is pretty stupid, but oh well... Change-Id: I1e605b2858a917f61922df757809a2aceb14a914
* update the gtk updater icon with the new updater iconMarkus Mohrhard2017-08-291-254/+200
| | | | Change-Id: I658a1ec54e74fe57efcaec873991d66be7ba31c1
* update updater iconandreas kainz2017-08-283-0/+649
| | | | | | | Change-Id: Ie4f38a51e87f29c322ba5aac2a2ad705bae42e71 Reviewed-on: https://gerrit.libreoffice.org/41660 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: store the updater GUI icon on linux in the codeMarkus Mohrhard2017-08-222-3/+261
| | | | | | The content can be created through convert from the png version. Change-Id: Icc07a118dfa139f270232a1e708e6907ad02ddea
* updater: get the updater GUI working on LinuxMarkus Mohrhard2017-08-224-4/+12
| | | | Change-Id: I3e7edb5319ab2e2ec316204b917a7e29c1791b5b
* updater: fix error in linux part of updater makefilesMarkus Mohrhard2017-08-221-1/+1
| | | | Change-Id: I2d32159364d34fb4154b7f88590045dc3a442bc3
* updater: add a way to test the dialog code of the updaterMarkus Mohrhard2017-08-213-0/+107
| | | | Change-Id: I7436edc85f87d1f68b50d39bf29564ff498c9ab9
* updater: always show the updater UIMarkus Mohrhard2017-08-211-3/+8
| | | | Change-Id: I0e1dc86673779966ff247b81852fb2f29655d2b9
* updater: set executable type GUI for updater.exeMarkus Mohrhard2017-08-211-2/+8
| | | | Change-Id: I795cbd8634a3afe219db033ce40f833af0eee44a
* updater: reference the correct new static library nameMarkus Mohrhard2017-08-141-1/+1
| | | | | | | Change-Id: Ib5920b27429f8414db761387645713498a60904f Reviewed-on: https://gerrit.libreoffice.org/41142 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: move the windows process starting code to comphelperMarkus Mohrhard2017-08-086-322/+3
| | | | | | | Change-Id: I1a499f57d01ee28afdb2c4f85dc976f2e6837dfd Reviewed-on: https://gerrit.libreoffice.org/40837 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: add include guard for windows process start codeMarkus Mohrhard2017-08-081-0/+5
| | | | | | | Change-Id: If021a92163381f2221faa1c7121f6c4af2d51dcf Reviewed-on: https://gerrit.libreoffice.org/40836 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: use the correct define checkMarkus Mohrhard2017-08-081-3/+1
| | | | | | | Change-Id: I79f47da401732e39bbce778d5b3971d6024a7935 Reviewed-on: https://gerrit.libreoffice.org/40835 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: disable the special handling of the user profileMarkus Mohrhard2017-08-021-0/+3
| | | | | | | | | This should be unnecessary for the one step updating strategy. Change-Id: I8dddeae4fb4a560a1c261c0c2e84183d72fc638c Reviewed-on: https://gerrit.libreoffice.org/40656 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* make it easier to debug the generation of certificate dataMarkus Mohrhard2017-08-011-2/+12
| | | | | | | Change-Id: I86e3c3254cef297c46a8ca377b5ceb36b348e2b3 Reviewed-on: https://gerrit.libreoffice.org/40607 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: get the signature verification working on WindowsMarkus Mohrhard2017-07-318-30/+77
| | | | | | | | | | | The updater itself does not link to NSS on Windows and instead uses the Windows-internal crypto libraries. However the mar executable needs to link against NSS, so we need to split the code a bit. Change-Id: Iea61c716aa507a876fdb444eb28ad828bdea18bd Reviewed-on: https://gerrit.libreoffice.org/40608 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: fix windows build, string issuesMarkus Mohrhard2017-07-311-2/+2
| | | | | | | Change-Id: I79ff08534dc522ede50cd0c5771576d825977b7c Reviewed-on: https://gerrit.libreoffice.org/40606 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: fix include and exception support for updater serviceMarkus Mohrhard2017-07-311-1/+2
| | | | | | | Change-Id: Ia0c7a62779bb8e870e8d5e458b81b5d2a09300d0 Reviewed-on: https://gerrit.libreoffice.org/40605 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* fix updater code on windowsMarkus Mohrhard2017-07-313-5/+199
| | | | | | | Change-Id: Ic1e39bc8f4ef6afe8781ce62997201acba56680a Reviewed-on: https://gerrit.libreoffice.org/40604 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: remove unnecessary mozilla makefileMarkus Mohrhard2017-07-281-9/+0
| | | | | | | Change-Id: Ia0162482841854cc4cb5c2b004bc978809bdf4c2 Reviewed-on: https://gerrit.libreoffice.org/40534 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* updater: make it more difficult to misuse the APIMarkus Mohrhard2017-06-261-29/+19
| | | | Change-Id: Ief5b2e37f289d6bfa92a909701c331d50509b0a3
* updater: reduce scope of ArchiveReader and pass by referenceMarkus Mohrhard2017-06-261-53/+48
| | | | Change-Id: Ifbcfa5b612cb0a8670bf3dd0be6e74708cc969c1
* updater: subclasses need to pass the archiveReader parameter to their parentsMarkus Mohrhard2017-06-261-11/+26
| | | | Change-Id: I1d69d592a05908e4e2a2e832a5b6ef9d13fc2e2c
* tdf#108563 Transform global variable gArchiveReader to local variableChristian Barth2017-06-261-23/+46
| | | | | Change-Id: I0830f00bce19bc4b77d8e9e0cb5684338a4c9e5d Signed-off-by: Christian Barth <Christian.Barth@zoho.com>
* updater: add a one step update testMarkus Mohrhard2017-06-244-0/+27
| | | | Change-Id: I548504ecc5fd2403596ad6eb5af116e4a46ddc35
* new cannot return nullptrNoel Grandin2017-06-201-4/+0
| | | | | | | | | unless we are using std::nothrow Change-Id: I3bdd13c8ce18f4e977f18ee5196311bf4fc62de0 Reviewed-on: https://gerrit.libreoffice.org/38998 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* -Werror=maybe-uninitializedMarkus Mohrhard2017-06-171-1/+1
| | | | Change-Id: Iba942ded86c8e4565bcd14796f05f47e3fc64880
* updater: fix missing header for std::vectorMarkus Mohrhard2017-06-171-0/+1
| | | | Change-Id: Ifd71745ce671a673ff3d4e9c2ca7a4abb6bed363
* updater: get the certificate code working on python2 and 3Markus Mohrhard2017-06-171-5/+12
| | | | Change-Id: Id1b4a443629c95fdbda59153c6f688629f6b1862
* updater: remove unnecessary fileMarkus Mohrhard2017-06-171-1/+0
| | | | Change-Id: I50d4ec48599d64db490b1dcac839a52737d105d9
* updater: remove some of the old mozilla makefile piecesMarkus Mohrhard2017-06-172-43/+0
| | | | Change-Id: Ib94c2a5e8335732823cd6632937ae77774a6746d
* updater: remove old updater certificate dataMarkus Mohrhard2017-06-171-0/+0
| | | | Change-Id: I5e47337682b5a887e544cbc7e7963570253a70fc
* updater: move the data location for the replacement testMarkus Mohrhard2017-06-172-4/+7
| | | | Change-Id: I6b3bbe9dda2461ed83ef8b4e15179ad345c08bd4
* updater: respect version info for manual testsMarkus Mohrhard2017-06-172-0/+0
| | | | Change-Id: Id9df2e685b7a7ff420bb7289089dc1e5c5f5c074
* updater: add manual test for new language updatesMarkus Mohrhard2017-06-164-0/+21
| | | | Change-Id: I6d77039f0cafe0f436cc687bb424588a59c3d464