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

python2Packages.legitfs: init at 0.3 #22137

Closed
wants to merge 2 commits into from
Closed

Conversation

layus
Copy link
Member

@layus layus commented Jan 25, 2017

No description provided.

Copy link
Member

@FRidh FRidh left a comment

Choose a reason for hiding this comment

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

tests aren't run

@FRidh
Copy link
Member

FRidh commented Jan 26, 2017

Also, will you maintain this package?

@layus
Copy link
Member Author

layus commented Jan 26, 2017

@FRidh I just noticed that version 0.4 was released at the same time as this PR.

This updated version

  • Has myself as a maintainer.
  • Uses version 0.4.0
  • Updates fastimport for py3k support (as legitfs added support for py3k in the meantime).
  • Still runs no tests, because legitfs has none

@globin
Copy link
Member

globin commented Jan 26, 2017

Could you change the message of the first commit python2Packages -> pythonPackages?

@FRidh
Copy link
Member

FRidh commented Jan 26, 2017

Still runs no tests, because legitfs has none

Then it needs doCheck = false; along with a comment stating it doesn't include tests.

@layus
Copy link
Member Author

layus commented Jan 30, 2017

Closing. the updated version does not work properly. I will resubmit later when I get more time to investigate.

@globin, @FRidh : Thanks for the guidelines about formatting & co in pythonPacakges.

@layus layus closed this Jan 30, 2017
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