search

latte-int / latte

LattE integrale, software for counting lattice points and integration over convex polytopes
https://www.math.ucdavis.edu/~latte/
GNU General Public License v2.0
39 stars 13 forks source link

header in code/latte/sqlite/ are not shipped in the released tarball #31

Closed kiwifb closed 2 years ago

kiwifb commented 2 years ago

I am guessing it is not a big deal because that stuff does not seem to be used/compiled at the moment.

So, I was busy making a new ebuild of latte-int for Gentoo that includes the commit 6dbf7f07d5c9e1f3afe793f782d191d4465088ae to fix issue #24. And then the patch failed because it could not find the file code/latte/sqlite/IntegrationDB.h to patch. And it could not find the file because it is not in the tarball for 1.7.6 that I am downloading here.

mkoeppe commented 2 years ago

I think leaving this code out from the tarball was the intention. This is just special purpose code that we needed for computations in a paper.

I did mess up https://trac.sagemath.org/ticket/33981, which broke that latte build in Sage...

kiwifb commented 2 years ago

I do warn people against doing that kind of mixing up "professionally" (I guess that's one of the thing I am actually paid for). But I had a feel that the whole folder did not belong.

mkoeppe commented 2 years ago

Did I mention there used to be a > 4GB database blob in the repo that I had to surgically remove at some point?

kiwifb commented 2 years ago

Did I mention there used to be a > 4GB database blob in the repo that I had to surgically remove at some point?

"I am not sure I want to know" is my standard answer to that kind of things.