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

Docs say not to use --offline, but not why #141

Closed
ringerc opened this issue Jul 30, 2014 · 5 comments
Closed

Docs say not to use --offline, but not why #141

ringerc opened this issue Jul 30, 2014 · 5 comments

Comments

@ringerc
Copy link
Contributor

ringerc commented Jul 30, 2014

The README.md says that --offline is intended to allow use of a local-only git repo without needing to hit a remote server for every action.

However, the asciidoc says:

--offline::
do not attempt any remote communication (avoid using
this please)

Why? Is this just outdated? Is there something broken that needs fixing?

@ringerc
Copy link
Contributor Author

ringerc commented Jul 30, 2014

tito build --help includes the same text.

@dgoodwin
Copy link
Collaborator

Offline lets you potentially do builds you can't reproduce, normally we check that the tag you're building exists in the remote repo. If it hasn't been pushed, it's quite easy to do a build that will result in a checksum that is no longer the same. This is something we try to avoid.

@ringerc
Copy link
Contributor Author

ringerc commented Jul 30, 2014

On 07/30/2014 10:31 PM, Devan Goodwin wrote:

Offline lets you potentially do builds you can't reproduce, normally we
check that the tag you're building exists in the remote repo. If it
hasn't been pushed, it's quite easy to do a build that will result in a
checksum that is no longer the same. This is something we try to avoid.

Ah, OK. So it works, but isn't recommended because it may lead to the
same tarball being produced with different checksums if the user hasn't
ensured everything's in sync with a central master repo.

Makes sense. It might be worth adding what you wrote above to the docs;
currently it's most unclear.

Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

@dgoodwin
Copy link
Collaborator

Yes absolutely, it's a bit longwinded for optparse help but it would be a good fit in man page.

xsuchy added a commit to xsuchy/tito that referenced this issue Apr 26, 2015
dgoodwin added a commit that referenced this issue May 14, 2015
clarify --offline option #141
@stevekuznetsov
Copy link
Collaborator

Fixed in xsuchy@f5f9e22

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

No branches or pull requests

3 participants