-
Notifications
You must be signed in to change notification settings - Fork 793
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
Feature/macos legacy #6498
Feature/macos legacy #6498
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (1/1)
717bc71
to
0310637
Compare
lgtm, does it compile on the build machine? I would have @claucambra here anyways for an expert look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the pbxproj changes should be reverted or amended, but everything else looks great :)
shell_integration/MacOSX/NextcloudIntegration/NextcloudIntegration.xcodeproj/project.pbxproj
Outdated
Show resolved
Hide resolved
0310637
to
3f9258c
Compare
Can we then backport this to 3.12? |
Signed-off-by: Camila <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
…pending on deployment version Signed-off-by: Claudio Cambra <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
Signed-off-by: Claudio Cambra <[email protected]>
It should now only use the CMAKE_OSX_DEPLOYMENT_TARGET set when building. Signed-off-by: Camila Ayres <[email protected]>
3f9258c
to
d86abd4
Compare
/backport to stable-3.12 |
AppImage file: nextcloud-PR-6498-d86abd4275dd1375b9d146bdb244d636306e08c2-x86_64.AppImage |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
The backport to # Switch to the target branch and update it
git checkout stable-3.12
git pull origin stable-3.12
# Create the new backport branch
git checkout -b backport/6498/stable-3.12
# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 7f57692a ed879511 3124dbe5 38e22e85 b6b174a4 1348e8aa d86abd42
# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/6498/stable-3.12 Error: Failed to cherry pick commits: error: no cherry-pick or revert in progress Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports. |
Much love from me for this effort! Thanks |
Trying to build mac OS legacy versions from master.