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
maxx: init at 1.1.0 #27096
maxx: init at 1.1.0 #27096
Conversation
pkgs/desktops/maxx/default.nix
Outdated
|
||
buildPhase = '' | ||
while IFS= read -r -d $'\0' i; do | ||
if ! isELF "$i"; then continue; fi |
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.
Please just indent the if block, remove the ! and continue.
pkgs/desktops/maxx/default.nix
Outdated
done < <(find "." -type f -print0) | ||
|
||
substituteInPlace bin/adminterm \ | ||
--replace /bin/bash /bin/sh |
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.
bash and sh are programs which interpret different languages, so you are potentially introducing a bug here.
If the upstream program is really a sh program, then you should use ${stdenv.shell}, and inform upstream to fix their system.
|
||
srcs = [ | ||
(fetchurl { | ||
url = "http://maxxinteractive.com/downloads/${version}/FEDORA/MaXX-${version}-NO-ARCH.tar.gz"; |
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.
Please ask upstream to provide sources in a location not mentioning the name of a Linux distribution.
pkgs/desktops/maxx/default.nix
Outdated
|
||
installPhase = '' | ||
maxx=$out/opt/MaXX | ||
mkdir -p $maxx $out/share |
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.
"$maxx"
pkgs/desktops/maxx/default.nix
Outdated
maxx=$out/opt/MaXX | ||
mkdir -p $maxx $out/share | ||
|
||
mv * $maxx |
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.
mv -- ./* "$maxx"
pkgs/desktops/maxx/default.nix
Outdated
longDescription = '' | ||
The MaXX Interactive Desktop a.k.a. The MaXX Desktop is the continuation | ||
of the 5dwm.org implementation released many years back. So don’t be | ||
mistaken, there is only one implementation of the SGI Desktop on Linux. |
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.
This is fairly confusing for someone who has no idea what this sub culture is about. If you want to say this is not some official implementation, then use the word "official" somewhere to stress that this isn't that.
pkgs/desktops/maxx/default.nix
Outdated
The MaXX Interactive Desktop a.k.a. The MaXX Desktop is the continuation | ||
of the 5dwm.org implementation released many years back. So don’t be | ||
mistaken, there is only one implementation of the SGI Desktop on Linux. | ||
Our goal is to bring to the masses this great user experience which focus |
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.
Filling long descriptions is nice, but it's also not intended as a Wikipedia page, AFAIK. So, please be a bit more compact.
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.
There are still some missing double quotes, but otherwise this is ready. I would be surprised if the number of users of this thing in NixOS is going to be higher than 1 in the next 5 years.
@Mic92 Please merge this. It's not important enough to discuss pretty much anything about it (because there are no known users(!)). |
@Mic92 The package has been updated to work on other distros in addition to NixOS. However, this link need to be created manually there:
Is it possible to create it automatically somehow? In NixOS I just use |
oh, this looks pretty ugly. I think it would be better to redirecting reads to |
@Mic92 Cool it helped, thanks :-) |
Thanks! |
Motivation for this change
Add MaXX - retro desktop environment
http://www.maxxinteractive.com/
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)