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

Add docker support, new param is introduced --docker-image #6

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

agustinhenze
Copy link
Contributor

e.g. piuparts --docker-image debian:unstable package.deb

It only supports overlay2 for now and it uses the MergedDir layer
where piuparts can access, add, edit and remove files easily.

@agustinhenze agustinhenze force-pushed the docker-support branch 2 times, most recently from 02567cb to 873e3a3 Compare March 22, 2018 12:24
piuparts.1.txt Outdated
Use schroot session named SCHROOT-NAME for the chroot, instead of building a new one with debootstrap.
Use schroot session named SCHROOT-NAME for the testing environment.

*--docker-image='DOCKER-IMAGE'::
Copy link

@inakimalerba inakimalerba Mar 22, 2018

Choose a reason for hiding this comment

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

It's missing a * to close the bold section after --docker-image

@inakimalerba
Copy link

Maybe you should add docker as a suggested package on the debian/control file ?

@anbe42
Copy link
Contributor

anbe42 commented Mar 22, 2018

@agustinhenze I'm probably not going to test or use it ... but looks good from a quick glance.
debian/control: Package piuparts should get a Suggests: docker, too.
And you should include a debian/changelog entry as well, closing the bug.

@agustinhenze
Copy link
Contributor Author

@anbe42 Thank you for doing review :). Please take a look again to see if everything is ok for you

@anbe42
Copy link
Contributor

anbe42 commented Mar 22, 2018

@agustinhenze Please rebase the pull request on develop, not master. Targeting develop not master. Make the changelog entry a multimaint one. (with [ Ag.. He.. ] header, don't touch the footer)

@agustinhenze agustinhenze changed the base branch from master to develop March 22, 2018 15:40
@agustinhenze
Copy link
Contributor Author

@anbe42 done

@anbe42
Copy link
Contributor

anbe42 commented Mar 23, 2018

ACK

@agustinhenze agustinhenze force-pushed the docker-support branch 2 times, most recently from 2292e3a to 9800785 Compare March 24, 2018 16:05
e.g. piuparts --docker-image debian:unstable package.deb

It only supports overlay2 for now and it uses the `MergedDir` layer
where piuparts can access, add, edit and remove files easily.

Signed-off-by: Agustin Henze <[email protected]>
Signed-off-by: Agustin Henze <[email protected]>
Signed-off-by: Agustin Henze <[email protected]>
Signed-off-by: Agustin Henze <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants