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

git placeholders are not expanded in release tarballs #499

Open
ryandesign opened this issue Aug 12, 2024 · 2 comments
Open

git placeholders are not expanded in release tarballs #499

ryandesign opened this issue Aug 12, 2024 · 2 comments

Comments

@ryandesign
Copy link

git placeholders are not expanded in the libqb-2.0.8.tar.xz and libqb-2.0.8.tar.gz release tarballs, suggesting that they were not created by git archive.

For example, if I use the automatically-generated v2.0.8.tar.gz tarball that I know from #312 I'm not supposed to use, this line:

m4_esyscmd([build-aux/git-version-gen $(echo '$Format:%h??%D$'\

is expanded to:

	m4_esyscmd([build-aux/git-version-gen $(echo '002171bbc??tag: v2.0.8'\

and this line:

SOURCE_EPOCH='$Format:%ct$' # template for rewriting by git-archive

is expanded to:

		 SOURCE_EPOCH='1689939039' # template for rewriting by git-archive

However in the release tarballs these $Format:…$ placeholders remain unchanged.

@chrissie-c
Copy link
Contributor

Hi, thanks for this.

The code does actually work, there's a check for 'Format:' in a case statement later in configure and the SOURCE_EPOCH is being updated through that. Whether this is the intent I'm not 100% sure as that code is copied from kronosnet and I didn't originate it. I'll consult with Fabio when he returns from holiday next week.

In the meantime I don't think it's a problem though.

@jfriesse
Copy link
Member

release.mk is using distcheck makefile target, not git archive. So it is expected git placeholders remain untouched. Same behavior is across libqb/knet/corosync/corosync-qdevice so if we decide to fix it we should fix it in all projects. The question is if this is really problem or not, because git placeholders are used to overcome problems with git archive generated archives (mostly version number) and such problems are not present in officially released tarballs created by release.mk.

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