8000 make.sh: leave around the generated version by vbatts · Pull Request #11651 · moby/moby · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

make.sh: leave around the generated version #11651

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

Merged
merged 1 commit into from
Mar 24, 2015

Conversation

vbatts
Copy link
Contributor
@vbatts vbatts commented Mar 23, 2015

For positerity (largely of packagers) lets leave around the generated
version files that happen during build.
They're already ignored in git, and recreated on every build.

ping @tianon and @lsm5

Signed-off-by: Vincent Batts vbatts@redhat.com

For positerity (largely of packagers) lets leave around the generated
version files that happen during build.
They're already ignored in git, and recreated on every build.

Signed-off-by: Vincent Batts <vbatts@redhat.com>
@lsm5
Copy link
Contributor
lsm5 commented Mar 23, 2015

LGTM, this does help me build https://github.com/vbatts/docker-utils .

@duglin
Copy link
Contributor
duglin commented Mar 23, 2015

ping @brahmaroutu

@tianon
Copy link
Member
tianon commented Mar 23, 2015

This is essentially a revert of #10957, although that PR is sparse on the details of why it was opened (besides being "generally OK"). 😞

@tianon
8000 Copy link
Member
tianon commented Mar 23, 2015

I feel like there was a specific problem this was solving, but I can't remember what it was. 😢

@brahmaroutu
Copy link
Contributor

Generally, it is OK to leave them around. LGTM

@crosbymichael
Copy link
Contributor

LGTM

crosbymichael added a commit that referenced this pull request Mar 24, 2015
make.sh: leave around the generated version
@crosbymichael crosbymichael merged commit 46be6bb into moby:master Mar 24, 2015
@jessfraz
Copy link
Contributor

does this need to be in 1.6.0

@vbatts
Copy link
Contributor Author
vbatts commented Mar 25, 2015

I'd like to include it so the version is present for posterity in packages.
On Mar 24, 2015 7:38 PM, "Jessie Frazelle" notifications@github.com wrote:

does this need to be in 1.6.0


Reply to this email directly or view it on GitHub
#11651 (comment).

@tianon
Copy link
Member
tianon commented Mar 25, 2015 via email

@vbatts
Copy link
Contributor Author
vbatts commented Mar 25, 2015

@tianon it is not to be confused with "orig" source, but like the btrfs
version.h that is a product of build and shipped in the -devel package.
On Mar 24, 2015 8:43 PM, "Tianon Gravi" notifications@github.com wrote:

It's interesting to me that RedHat can (and prefers to) include build
artifacts like this in their "source" representations. For Debian, the
focus is on the pristine, un-artifacted upstream source tarball (the "orig"
tarball), so build artifacts like this don't matter, especially since
they're trivially generated from the actual source code from upstream.
(Just as some extra context for why I'm confused about this being good for
the packaging.)


Reply to this email directly or view it on GitHub
#11651 (comment).

@jessfraz jessfraz added this to the 1.6.0 milestone Mar 25, 2015
@tianon
Copy link
Member
tianon commented Mar 25, 2015 via email

@vbatts vbatts deleted the vbatts-dont_rm_dockerversion branch April 27, 2016 18:03
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.

8 participants
0