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

[regression] build.sh now requires sudo? #287

Closed
lasconic opened this issue Nov 15, 2016 · 6 comments
Closed

[regression] build.sh now requires sudo? #287

lasconic opened this issue Nov 15, 2016 · 6 comments
Labels

Comments

@lasconic
Copy link

lasconic commented Nov 15, 2016

Since 12 days or so (around november 3 or 4), the master branch fails to create an AppImage for MuseScore.

See https://travis-ci.org/musescore/MuseScore/jobs/175875703#L1081

Making all in man
make[2]: Entering directory `/AppImageKit/inotify-tools-3.14/man'
make[3]: Entering directory `/AppImageKit/inotify-tools-3.14'
make[3]: Leaving directory `/AppImageKit/inotify-tools-3.14'
make[2]: Nothing to be done for `all'.
make[2]: Leaving directory `/AppImageKit/inotify-tools-3.14/man'
make[2]: Entering directory `/AppImageKit/inotify-tools-3.14'
cd . && /bin/sh ./config.status config.h
config.status: creating config.h
make[2]: Leaving directory `/AppImageKit/inotify-tools-3.14'
make[1]: Leaving directory `/AppImageKit/inotify-tools-3.14'
+ sudo make install
/opt/rh/devtoolset-2/root/usr/bin/sudo: line 14: /usr/bin/sudo: No such file or directory
/opt/rh/devtoolset-2/root/usr/bin/sudo: line 14: exec: /usr/bin/sudo: cannot execute: No such file or directory
@probonopd
Copy link
Member

Hi @lasconic, are you talking about the master branch of AppImageKit? I did not intentionally change something there. From the snippet it looks like you are using the (new) appimagetool/master branch of AppImageKit which indeed has changed in significant ways. Are you sure you need to build AppImageKit yourself on each build? I'd recommend you to consider using the (new) appimagetool/master branch of AppImageKit but store a "known good" version of appimagetool.AppImage which you can then use as part of your build process.

@probonopd
Copy link
Member

probonopd commented Nov 15, 2016

Some background information about the appimagetool/master branch:
http://discourse.appimage.org/t/the-future-of-appimage-type-2-and-new-tools

@probonopd
Copy link
Member

@lasconic were you able to try appimagetool? Let me know if it works for you please.

@lasconic
Copy link
Author

lasconic commented Dec 3, 2016

@shoogle is looking into it.

@shoogle
Copy link

shoogle commented Dec 4, 2016

076a88a solves it for us. I'll put this in a Docker image and investigate appimagetool later.

@azubieta
Copy link
Member

build.sh no longer requires sudo. Closing the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants