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
scilab-bin: adding application launchers #51363
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.
Can you separate your commands, with comments, into groups like:
- moving desktop files
- moving icons
- correcting content of desktop files
Without that visual hierarchy (or something similar) it's kind of thick to read.
Also why not using substituteInPlace
instead of sed directly?
Largely used sed as I know how to use it, I don't know how to use substituteInPlace. I'll add the comments. |
substituteInPlace usually looks like: substituteInPlace file \
--replace pattern replacement Though it isn't a problem if you don't use it. |
35a4da6
to
8481f81
Compare
I think so. I'm verifying locally now. |
My last commit is essentially me just making things more concise. |
$out/opt/scilab-6.0.1/bin/scilab-cli doesn't execute and by extension the desktop file doesn't launch the application. |
Aha, lol, forgot to create the needed symlink. |
While I'm at it, do you know why there's |
I don't really know. You could try removing them and see if it works. Git blame reveals that its always been here. |
ae71151
to
725912e
Compare
Btw, executing
So I guess the bash script Possibly something to fix later. |
Ah, that is unexpected. |
Would you mind moving |
Moving the desktop configuration files from opt/scilab-$ver/share/applications to share/applications, making the required changes, and moving the icons in opt/scilab-$ver/share/icons to share/icons. Moving locales, appdata and mime out of there opt/scilab-${ver} too. Also removing unneeded scilab-${ver} and scilab-${majorVer} symlinks from bin/.
Now I'm going to check if it builds FTR @GrahamcOfBorg build scilab-bin |
@fusion809 Thanks for contributing ✨ |
Moving the desktop configuration files from
opt/scilab-$ver/share/applications
toshare/applications
, making the required changes, and moving the icons inopt/scilab-$ver/share/icons
toshare/icons
.Motivation for this change
Presently the scilab-bin package provides no application launchers for the Scilab, Scinotes and Xcos apps. In this PR I am adding these launchers.
On lines 76, 79 and 82 I am replacing
Terminal=false
withTerminal=true
in the Scilab, Scinotes and Xcos desktop config files, as I've tried launching these apps using these launchers, without theTerminal=true
change, and they always crash on start up. Changing toTerminal=true
fixes this problem for all three apps.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)