openSeaChest icon indicating copy to clipboard operation
openSeaChest copied to clipboard

release tarballs aren't buildable

Open asomers opened this issue 7 years ago • 2 comments

The release tarballs don't include the git submodules, which makes them unbuildable. Tested on OpenSeaChest 18.04 with the release tarball posted on GitHub.

asomers avatar Sep 26 '18 23:09 asomers

We are working on a 18.10 release which will be finished before the end of this month and it should have all the FreeBSD fixes.

xahmad avatar Oct 04 '18 16:10 xahmad

The problem with the release tarballs isn't FreeBSD-specific; they simply don't include all of the code. The tarballs, as opposed to the git checkout, don't build anywhere.

asomers avatar Oct 04 '18 16:10 asomers

ping

thesamesam avatar Jan 02 '23 19:01 thesamesam

I added a new action that will create a Source_With_Submodules.tar.xz and Source_With_Submodules.zip when we push release tags.

It seems that the default for a release uses git archive which does not traverse submodules. I read up on this issue (multiple git repos on Github have similar issues filed) and it seems git-archive-all is the best solution to use, so this new Github action will use that to publish these at the releases going forward.

vonericsen avatar Jan 06 '23 00:01 vonericsen

Thank you!

thesamesam avatar Jan 06 '23 00:01 thesamesam

I see the new tarball in the alpha release https://github.com/Seagate/openSeaChest/releases/tag/v23.02-alpha.1, and it looks to build just fine for me. I think it is a little weird that the folder extracts to Source_With_Submodules. and I'd maybe prefer it extract to openSeaChest-23.02-alpha.1 to match the release version. But that's just stylistic, it doesn't really matter. Thanks for adding this, it'll save me a lot of trouble!

faithanalog avatar Feb 02 '23 08:02 faithanalog

@faithanalog I will investigate changing that to the version you mentioned. I think that would make a lot more sense.

vonericsen avatar Feb 02 '23 16:02 vonericsen

This has been fixed in https://github.com/Seagate/openSeaChest/releases/tag/v23.03 I even got the subfolder to have a nice name like @faithanalog mentioned previously! 😄

If you have any trouble, please feel free to reopen this issue and I will look at this again.

vonericsen avatar Mar 01 '23 21:03 vonericsen