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
bless: init at 0.6.2 #75670
bless: init at 0.6.2 #75670
Conversation
I've noticed this has several issues when running it on nixOS, didn't notice them at first because I had everything installed on my ubuntu machine. Need to fix those first |
# and it also wants to install that file | ||
touch ./doc/user/bless-manual.omf | ||
|
||
./autogen.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.
Does not preAutoreconf
work?
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.
I also need to invoke ./autogen.sh because the autoreconfHook calls the tools directly, but the script invokes some more tools that create necessary files.
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.
And it seems to work that way, because the prefix is passed via configure phase
Ook, I was already looking forward to removing gtk-sharp. Yeah, also getting some errors: $ result/bin/bless
result/bin/bless: line 3: mono: command not found
$ nix run -f '<nixpkgs>' mono -c result/bin/bless
Unhandled Exception:
System.TypeLoadException: Could not load type of field 'BlessMain:DataViewBox' (0) due to: Could not load file or assembly 'gtk-sharp, Version=2.12.0.0, Culture=neutral, PublicKeyToken=35e10195dab3c99f' or one of its dependencies.
[ERROR] FATAL UNHANDLED EXCEPTION: System.TypeLoadException: Could not load type of field 'BlessMain:DataViewBox' (0) due to: Could not load file or assembly 'gtk-sharp, Version=2.12.0.0, Culture=neutral, PublicKeyToken=35e10195dab3c99f' or one of its dependencies. |
Seems like it tries to search for it in $mono and $bless |
It talks about LD_LIBRARY_PATH |
It's fixed, please re-review |
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.
nixpkgs-review
passes
executable works
NixOS unstable on x86_64
[3 built, 11 copied (332.9 MiB), 84.1 MiB DL]
https://github.com/NixOS/nixpkgs/pull/75670
1 package built:
bless
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Applied review changes, should be good to merge now |
/me begs for a merge |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Motivation for this change
Needed this for myself, might be useful for others, too
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @