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

jail git support documentation #279

Closed
wants to merge 9 commits into from

Conversation

JamesKoenig
Copy link

added documentation requested by bdrewery on splbio's pull request to add git support to poudriere's jail subcommand (#258). This fork's jail subcommand was tested and works.

@bdrewery bdrewery changed the title 3.1.1 git Add git support for jail command Mar 22, 2015
@bdrewery bdrewery changed the title Add git support for jail command jail git subcommand documentation Mar 22, 2015
@bdrewery bdrewery changed the title jail git subcommand documentation jail git support documentation Mar 22, 2015
@JamesKoenig
Copy link
Author

if you don't mind holding off on this, I'm going to try and clean this up so it's more like the rest of poudriere's behavior (eg. the behavior in this merge request so far for poudriere jail & git, vs the behavior currently in poudriere ports for git), like allowing SCM_URL to be set in poudriere.conf (instead of explicitly on the command line), and reconciling the disparity between SCM_URL in jail.sh vs GIT_URL in ports.sh.

…ther, in a customizable way, in regards to git (that is, added git url specification with -U to ports.sh, changed branch setting from -b to -B in jail.sh), & removed the now-unused 'proto' variable and the case statment related to it, from jail.sh's install_from_git function

Sponsored by: Norse Corporation
@JamesKoenig
Copy link
Author

I rebased off master, to save work, which means I need to close this pull request & resubmit it with the new (rebased) branch.

@JamesKoenig JamesKoenig closed this Apr 1, 2015
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.

2 participants