-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
lowdown: init at 0.7.2 #94225
lowdown: init at 0.7.2 #94225
Conversation
@GrahamcOfBorg build lowdown |
|
||
postInstall = '' | ||
mkdir -p ''${!outputDevman}/share/man | ||
mv ''${!outputMan}/share/man/man3 ''${!outputDevman}/share/man/ |
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 wouldn’t bother about devman
, it’s just more overhead for little gain (plus it’s confusing when the manpage is missing).
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
manual says "$outputMan
is for man pages (except for section 3)."
Maybe it's clearer, if we install user man pages to out
directly, man3
to devMan
and drop the man
output altogether?
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 used outputs = [ "out" "dev" ]
, any other outputs are overkill given their sizes. For example, lib
only contains a static library so it's not useful to have it separate from dev
.
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.
Seems sane, but this way all manpages end up in out
, since a fixup hook moves them there. Might be confusing if the dev docs are shipped with the executables?
Motivation for this change
Add lowdown, a markdown converter plus library.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)