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
pythonPackages.flask-autoindex: init at 0.6 #38784
Conversation
meta = with stdenv.lib; { | ||
description = "The mod_autoindex for Flask"; | ||
longDescription = '' | ||
Flask-AutoIndex generates an index page for your Flask application automatically. The result just like mod_autoindex, but the look is more awesome! |
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 split into two lines.
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.
Is there a standard for longDescription
line length? In my opinion non-semantic linebreaks should be done by whatever renders the description.
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 don't know of any. But if there's a .
in your longDescription
, you should definitely add a line break.
f7853ad
to
8054d2a
Compare
description = "The mod_autoindex for Flask"; | ||
longDescription = '' | ||
Flask-AutoIndex generates an index page for your Flask application automatically. | ||
The result just like mod_autoindex, but the look is more awesome! |
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 think there's a verb missing
8054d2a
to
9f5c3b8
Compare
9f5c3b8
to
3b6a3f8
Compare
I rebased on master since the dependency is now merged. |
3b6a3f8
to
d9e303a
Compare
@GrahamcOfBorg build python2.pkgs.flask-autoindex python3.pkgs.flask-autoindex |
Success on x86_64-linux (full log) Attempted: python2.pkgs.flask-autoindex, python3.pkgs.flask-autoindex Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.flask-autoindex, python3.pkgs.flask-autoindex Partial log (click to expand)
|
postInstall = '' | ||
# binary does not run, known issue and fixed in next release (>0.6) | ||
# see https://github.com/sublee/flask-autoindex/pull/32 | ||
rm $out/bin/fai |
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 don't think there will be a next release. Maybe use https://github.com/sublee/flask-autoindex/pull/32.patch as a patch?
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.
You might be right. Done.
d9e303a
to
a8f9fc1
Compare
@GrahamcOfBorg build python2.pkgs.flask-autoindex python3.pkgs.flask-autoindex |
Success on x86_64-linux (full log) Attempted: python2.pkgs.flask-autoindex, python3.pkgs.flask-autoindex Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.flask-autoindex, python3.pkgs.flask-autoindex Partial log (click to expand)
|
Motivation for this change
Depends on #38783, do not merge before that.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)