Age | Commit message (Collapse) | Author |
|
It's a convention on macOS that app bundle names start with an uppercase
letter. Also this aligns the app name with Windows and Linux.
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Signed-off-by: Dmytro Korchynskyi <kdl.dima@gmail.com>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
This reverts commit 3000e2664068b2b9dbc57e3f1a53653e400593dc.
This was not intended for a direct push
Signed-off-by: Matthieu Gallien <matthieu_gallien@yahoo.fr>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Instead of relying on ECM being installed.
We copy other stuff from ECM so be consistent.
Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
|
|
Use the FindInotify from ECM. It handles the case of Linux vs BSD
correctly automatically.
Also only find Inotify on Linux and BSD
Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>
|
|
Signed-off-by: Elsie Hupp <9206310+elsiehupp@users.noreply.github.com>
|
|
Instead of using custom find modules.
This allows using imported targets which make the code much nicer.
Signed-off-by: Nicolas Fella <nicolas.fella@gmx.de>
|
|
|
|
|
|
|
|
Installing to lib/${APPLICATION_EXECUTABLE} has caused a bunch of
irritations in the past and subtle annoying to fix bugs. To avoid name
clashes with branded clients ${APPLICATION_EXECUTABLE} becomes now
part of the filename instead of the subfolder.
The concrete motivation to change this now is that on Windows there
is no RPATH and it's not possible to run owncloud directly from the
Craft Root folder, which is nice when you're developing on Windows.
It would have been possible to change this just for Windows but as
written earlier this has caused lots of issues and thus I think it's
a good idea to just stay consistent accross platforms when touching it.
|
|
Issue #6278
|
|
|
|
|
|
Also change the placeholder suffix config option to not include the dot,
the dotless form is needed in the nsis script.
|
|
|
|
QtKeychain provides Qt5KeychainConfig.cmake and friends nowadays, so no
need to have a less reliable and outdated find module on our end.
Also this shows that we were including keychain.h in the wrong way and
were not using the link target, so both got fixed as well.
Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>
|
|
Required for our build server that uses an older SDK to support previous macOS versions.
See: https://developer.apple.com/documentation/appkit/nsappearancecustomization/choosing_a_specific_appearance_for_your_macos_app
Signed-off-by: Michael Schuster <michael@schuster.ms>
|
|
The UndefinedBehaviorSanitizer includes the "vptr" check. This
check, however, needs typeinfo for OCC::AccountManager because
otherwise its stub for FileManTest leads to undefined references
when linking. Adding the -frtti flag to enable run-time typeinfo
did not solve the problem. I do not know another solution, so this
commit disables the vptr check.
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
|
|
The SANITIZE_ADDRESS option of our CMake configuration activates the
AddressSanitizer (and UBSan in a non-working way) for the whole project
(although, by the way, its documentation pretends that it is only enabled
for tests).
This commit introduces new options SANITIZE_LEAK, SANITIZE_MEMORY,
SANITIZE_UNDEFINED, SANITIZE_THREAD. Each of these options (including
SANITIZE_ADDRESS) enables only the corresponding sanitizer.
Moreover, we mark all sanitizer options as advanced options, because these
options are only interesting for developers.
Note that some sanitizers are conflicting, that is, not all options can
be enabled simultaneously. Also, not all sanitizers are available for
all compilers and versions. We, however, do not check for this, instead
we let the compiler throw its errors in such cases.
The explicit usage of the Google Linker is removed, because it is not
necessary and can lead to problems with clang.
The commit can be considered a rewrite of cmake/modules/SanitizerFlags.cmake.
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
|
|
Debian has a list of known typos in their checking tools.
Signed-off-by: Sandro Knauß <hefee@debian.org>
|
|
|
|
Signed-off-by: Michael Schuster <michael@schuster.ms>
|
|
Signed-off-by: asapelkin <asapelkin0x01@ya.ru>
|
|
Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
|
|
fromhttps://github.com/KDE/extra-cmake-modules/blob/master/modules/ECMAddAppIcon.cmake with modifications for nc workflow (incl. png2imagemagick)
Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
|
|
filenames to not change pattern in cmake scripts
Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
|
|
versions reg. the sizes) in https://github.com/KDE/extra-cmake-modules/blob/master/modules/ECMAddAppIcon.cmake
Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
|
|
|
|
Replaced ownCloud.icns with APPLICATION_ICON_NAME variable
|
|
Newer Windows versions are also displaying a 256x256 icon. If this icon size is not present, the 48x48 icon is used, but not resized. This means there is a possibly large empty area around the icon.
|
|
without that key, the client automatically runs on the dedicated gpu (if
present). now it allows the system to use the integrated one.
closes #501
|
|
|
|
|
|
The option -Wno-discarded-qualifiers only exists with GCC, clang warns that
it has no effects.
Also it warns when some virtual fuction are marked with override but not
all of them.
|
|
|
|
|
|
cmake/modules/ECMAddAppIcon.cmake is heavily patched to support sidebar icons, OUTFILE_BASE parameter and to not include 64 and 64@2x icons on macOS which are not supported. All changes are made in a way that we can upstream this.
|
|
|
|
It's needed since 586fd346ead9aaa21aace8b2ffd616f0cc249a9e activated stack-protector-strong
|
|
|
|
|
|
Use modern cmake with target_link_libraries and Qt5:: that
automatically add the include path and compile flags
|
|
This commit integrates support for libcloudproviders
desktop integration API. If build with the library it
will check on startup if the DBus interface is available
and then use it instead of the legacy status icon.
Signed-off-by: Julius Härtl <jus@bitgrid.net>
|