Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only build app bundle if requested #3983

Merged
merged 1 commit into from Nov 23, 2021

Conversation

FlexW
Copy link

@FlexW FlexW commented Nov 17, 2021

For development a app bundle is not needed. The app bundle is only
needed for distribution and macdeployqt takes a lot of time.

Signed-off-by: Felix Weilbach felix.weilbach@nextcloud.com

FlexW pushed a commit to nextcloud/client-building that referenced this pull request Nov 17, 2021
After nextcloud/desktop#3983 gets merged, this
variable needs to be set explicitly.

Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
Copy link
Collaborator

@mgallien mgallien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@mgallien
Copy link
Collaborator

/rebase

@nextcloud-command nextcloud-command force-pushed the bugfix/only-build-app-bundle-if-needed branch from bd7bebf to c43ee96 Compare November 17, 2021 13:31
@FlexW
Copy link
Author

FlexW commented Nov 17, 2021

Please make sure this one gets only merged after https://github.com/nextcloud-gmbh/brander/pull/516 was merged. Otherwise, Brander will break.

@camilasan
Copy link
Member

/rebase

For development a app bundle is not needed. The app bundle is only
needed for distribution and macdeployqt takes a lot of time.

Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
@nextcloud-command nextcloud-command force-pushed the bugfix/only-build-app-bundle-if-needed branch from c43ee96 to ec64246 Compare November 23, 2021 11:41
@sonarcloud
Copy link

sonarcloud bot commented Nov 23, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-3983-ec64246dc7f0f2b865ee2cb92f6996f15fd1eaaa-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@camilasan camilasan merged commit 79282a8 into master Nov 23, 2021
@camilasan camilasan deleted the bugfix/only-build-app-bundle-if-needed branch November 23, 2021 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants