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.

Sign up for GitHub

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

Jump to bottom

Keep-Alive #764

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Keep-Alive #764

wants to merge 13 commits into from

Conversation

gronke
Copy link
Member

@gronke gronke commented Nov 13, 2020

  • refactor requirement parsing in setup.py

@igalic
Copy link
Collaborator

igalic commented Nov 13, 2020

given that libioc is a library, we should python packaging best practices, and not have a requirements.txt, much less use unstable APIs to parse it

we could use the same approach i suggested in gronke/py-freebsd_sysctl#9 and add our requirements in setup.cfg, and get rid of a lot of useless code

igalic
igalic reviewed Nov 14, 2020
View reviewed changes
Copy link
Collaborator

@igalic igalic left a comment

Choose a reason for hiding this comment

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

🙋🏻‍♀️

setup.cfg Outdated Show resolved Hide resolved
@igalic igalic mentioned this pull request Nov 14, 2020
Open
igalic
igalic requested changes Nov 14, 2020
View reviewed changes
Copy link
Collaborator

@igalic igalic left a comment

Choose a reason for hiding this comment

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

🙋🏻‍♀️

setup.cfg Outdated
install_requires =
gitpython
freebsd_sysctl==0.0.7
jail==0.0.11
Copy link
Collaborator

Choose a reason for hiding this comment

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

this keeps us only one step away from the current setup
we should use ranges in libioc, and only in ioc use exact versions of the published (on ports) packages

Copy link
Member Author

Choose a reason for hiding this comment

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

I agree that major version pinning is sufficient. Meanwhile the interfaces of both tools are stable enough to not break.

Copy link
Collaborator

Choose a reason for hiding this comment

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

time for a 1.0 release then

gronke added a commit that referenced this pull request Jun 23, 2021
@gronke
add iocage/iocell to package keywords
af8c810 
addresses #764 (comment)
@igalic
Copy link
Collaborator

igalic commented Jul 21, 2021

why are the cirrus tests failing like that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@igalic igalic igalic requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gronke @igalic @thatch