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

fix #2619 #2620

Merged
merged 6 commits into from
Apr 15, 2024
Merged

fix #2619 #2620

merged 6 commits into from
Apr 15, 2024

Conversation

enoch85
Copy link
Member

@enoch85 enoch85 commented Apr 1, 2024

make sure the bin is exceutable

fix #2619

make sure the bin is exceutable

Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
@enoch85 enoch85 changed the title part fix #2619 fix #2619 Apr 1, 2024
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
nextcloud_update.sh Outdated Show resolved Hide resolved
nextcloud_update.sh Outdated Show resolved Hide resolved
nextcloud_update.sh Outdated Show resolved Hide resolved
enoch85 and others added 2 commits April 2, 2024 13:53
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
nextcloud_update.sh Outdated Show resolved Hide resolved
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
@enoch85
Copy link
Member Author

enoch85 commented Apr 15, 2024

@pjrobertson Any update? :)

@pjrobertson
Copy link
Contributor

Sorry, I was wary of running this on a production server, and don't have a staging server set up yet :'(

I did end up running it though, and this is what I got. I think this looks ok. The only one I'm not sure about - if it needs to be in there - is the Collabora_Online.AppImage?

++ find /var/www/nextcloud/apps -type f -executable
+ find_executables='./mail/vendor/christophwurst/kitinerary-bin/bin/kitinerary-extractor
./notify_push/bin/x86_64/notify_push
./richdocumentscode/collabora/Collabora_Online.AppImage
./translate/bin/node
./recognize/bin/node'

@enoch85
Copy link
Member Author

enoch85 commented Apr 15, 2024

LGTM, let's merge!

@enoch85 enoch85 merged commit 28b4b20 into master Apr 15, 2024
7 checks passed
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.

Executable files are not executable (+x) after upgrade
2 participants