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

3to2: init at 1.1.1 #25202

Merged
merged 3 commits into from May 16, 2017
Merged

3to2: init at 1.1.1 #25202

merged 3 commits into from May 16, 2017

Conversation

mt-caret
Copy link
Contributor

@mt-caret mt-caret commented Apr 25, 2017

Motivation for this change

add 3to2

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.

@mention-bot
Copy link

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

version = "1.1.1";
name = "${pname}-${version}";

src = 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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried fetchPypi, but I it 404s for some reason, I think it has something to do with the package name starting with a number instead of a character.

Copy link
Member

Choose a reason for hiding this comment

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

ah, no, they published a zip instead of a tar.gz. I still need to fix fetchPypi to try both since on PyPI you're allowed to upload either, but not both.

Copy link
Member

Choose a reason for hiding this comment

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

@FRidh Or just allow to override the default from tar.gz to zip by the package author. This is how it is done in OpenEmbedded.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@FRidh is there any update on adding support for zips for fetchPypi? I have another python package (fonttools) which only publishes zip files. If you're busy right now, I think I'll give it a try.

Copy link
Member

Choose a reason for hiding this comment

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

@mt-caret go give it a try!

'';

# Test failing due to upstream issue (https://bitbucket.org/amentajo/lib3to2/issues/50/testsuite-fails-with-new-python-35)
doCheck = false;
Copy link
Member

Choose a reason for hiding this comment

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

and if you run it with other versions of Python?

Copy link
Contributor Author

@mt-caret mt-caret Apr 25, 2017

Choose a reason for hiding this comment

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

nix-build -A python33Packages.3to2 gives me error: hypothesis-3.7.0 not supported for interpreter python3.3m

Copy link
Member

Choose a reason for hiding this comment

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

That's fine, you can ignore that.

Copy link
Member

Choose a reason for hiding this comment

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

and how is it when you it with Python 2.x?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nix-build -A python27Packages.3to2 fails tests exactly like the issue here: https://bitbucket.org/amentajo/lib3to2/issues/50/testsuite-fails-with-new-python-35

meta = {
homepage = https://bitbucket.org/amentajo/lib3to2;
description = "Refactors valid 3.x syntax into valid 2.x syntax, if a syntactical conversion is possible";
license = lib.licenses.asl20;
Copy link
Member

Choose a reason for hiding this comment

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

will you maintain this package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm pretty new to Nix, but if that's okay I have no problem with maintaining this.

Copy link
Member

Choose a reason for hiding this comment

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

That's how we all begin :)
It's basically just about keeping track of updates and making sure the expression keeps working.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds great, added a commit adding myself to the maintainer list.

@7c6f434c
Copy link
Member

7c6f434c commented May 1, 2017

@FRidh there are new commits (and GitHub doesn't auto-notify of that)

@FRidh FRidh self-assigned this May 1, 2017
@FRidh FRidh merged commit 1aaeb94 into NixOS:master May 16, 2017
@mt-caret mt-caret deleted the 3to2 branch May 17, 2017 11:01
@mt-caret mt-caret mentioned this pull request May 25, 2017
7 tasks
@eqyiel eqyiel mentioned this pull request Aug 21, 2017
8 tasks
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

5 participants