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

Added #!/bin/sh to binstall.sh to help Konqueror know it is a script and not a text file #415

Closed
romw opened this issue Feb 3, 2015 · 3 comments

Comments

@romw
Copy link
Member

romw commented Feb 3, 2015

Reported by Joses on 22 Dec 37788460 13:20 UTC
There are 3 things I noticed that help Linux KDE Konqueror realize that run_manager is a script and not a text file (if you click the link).

  1. the run_manager script needs to be marked as executable +x
  2. The text file needs to begin with $!/bin/sh
  3. The file appears to work if it ends in .sh

This diff file does !#2 but not !#3 since you probably have a lot of help-info referring to run_client and run_manager and not run_client.sh and run_manager.sh
If you want to do !#3 as well, I leave that to you depending on your experience.

KDE Konqueror referred to here would be including version 3.5.6 found in Mandriva 2007. Newer distros will/may likely have better knowledge and perhaps this diff file might not be necessary for them.

@romw
Copy link
Member Author

romw commented Feb 4, 2015

Commented by Nicolas on 8 Sep 37789674 04:00 UTC
Reformatted to avoid ticket links on description.

@romw
Copy link
Member Author

romw commented Feb 4, 2015

Commented by Joses on 6 Sep 37996868 09:46 UTC
I explained the !bin/sh for Konqueror as step#6 on my HowTo in case this doesn't get fixed in source ....to help users past this problem in the meantime. :-)
http://www.joescat.com/boinc/index.html#OPEN

@romw
Copy link
Member Author

romw commented Feb 4, 2015

Commented by ChristianB on 22 Jan 40704968 12:26 UTC
This stuff now moved to packages/generic/sea/Makefile.am and custom packages are available for several linux distribution. If the additions to binstall.sh are still needed, please reopen.

@AenBleidd AenBleidd closed this as not planned Won't fix, can't repro, duplicate, stale Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants