Skip to content
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.ablog: init at 0.8.4 #25732

Closed
wants to merge 1 commit into from
Closed

pythonPackages.ablog: init at 0.8.4 #25732

wants to merge 1 commit into from

Conversation

massimo-zaniboni
Copy link
Contributor

added also the necessary package

pythonPackages.invoke: init at 0.17.0

Motivation for this change

I need to use the package in production, so I added and tested it.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

added also the necessary package

pythonPackages.invoke: init at 0.17.0
@mention-bot
Copy link

@massimo-zaniboni, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer.

@FRidh
Copy link
Member

FRidh commented May 12, 2017

Please read the header of the file you modified.


propagatedBuildInputs = with self; [ sphinx alabaster invoke dateutil werkzeug ];

src = pkgs.fetchurl {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fetchPypi

meta = {
description = "A Sphinx extension that converts any documentation or personal website project into a full-fledged blog.";
license = licenses.mit;
homepage = http://ablog.readthedocs.org/;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you going to maintain this package?

@massimo-zaniboni
Copy link
Contributor Author

About travis error: I will fix the merge request within few days, applying this time all the necessary test commands, as listed in the error messages. So I hope to be 100% correct the next time. Sorry!

BTW I followed the instructions on http://nixos.org/nixpkgs/manual/#chap-submitting-changes but they are not in par with what I understood here/now. Maybe I will try to update the manual too.

@massimo-zaniboni
Copy link
Contributor Author

Please read the header of the file you modified.

ABlog is an application / Python cli tool so (if I understood correctly) I need to move away from this module that contains only libraries, and build with buildPythonApplication in the proper module for stand-alone applications.

BTW as partial excuse, in this module there is also Sphinx, from which ABlog is derived, and so I considered (faulty) as the right place. Sorry.

I will update within few days, and I will integrate also the other change requests.

@massimo-zaniboni
Copy link
Contributor Author

@FRidh I decided to abort the PR because I will not use ABlog in production, and probably this PR is not useful to others.

Many thanks for the review, and sorry for the waste of time. In any case I understood better something about Nix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants