rampion / ReadArgs

Easy reading and parsing of command line arguments
BSD 3-Clause "New" or "Revised" License
8 stars 6 forks source link

Please include ReadArgsSpec.hs in the tarball #4

Closed markwright closed 11 years ago

markwright commented 12 years ago

As we build the tests on Gentoo. Not sure why its not included, as when I do:

cabal sdist

it includes it. This is with:

cabal --version

cabal-install version 1.16.0.1 using version 1.16.0.2 of the Cabal library #

rampion commented 12 years ago

Weird! I'll look into it.

rampion commented 12 years ago

So, I haven't written a ton of packages. What tarball are you referring to? What cabal command should generate it?

markwright commented 12 years ago

/usr/portage/distfiles/ReadArgs-1.2.tar.gz is a copy of the one on hackage:

http://hackage.haskell.org/packages/archive/ReadArgs/1.2/ReadArgs-1.2.tar.gz

The command:

cabal sdist

in the repo is one way to generate the tarball. If any extra source files or data files are added to the tests, then they need to be added to extra-source-files or maybe other-modules in the Test-Suite ReadArgsSpec section in order to be include in the cabal sdist.

argus% git pull --rebase
Current branch master is up to date.
argus% git status
# On branch master
nothing to commit, working directory clean
argus% grep -i version ReadArgs.cabal 
Version:             1.2
Cabal-version:       >=1.8
argus% cabal sdist
Warning: Cannot run preprocessors. Run 'configure' command first.
Building source dist for ReadArgs-1.2...
Source tarball created: dist/ReadArgs-1.2.tar.gz
argus% tar tvf dist/ReadArgs-1.2.tar.gz 
drwxr-xr-x 0/0               0 2012-11-09 08:20 ReadArgs-1.2/
-rw-r--r-- 0/0           10206 2012-11-09 08:20 ReadArgs-1.2/ReadArgs.hs
-rw-r--r-- 0/0             542 2012-11-09 08:20 ReadArgs-1.2/ReadArgsEx.hs
-rw-r--r-- 0/0            4620 2012-11-09 08:20 ReadArgs-1.2/ReadArgsSpec.hs
-rw-r--r-- 0/0            1533 2012-11-09 08:20 ReadArgs-1.2/LICENSE
-rw-r--r-- 0/0              46 2012-11-09 08:20 ReadArgs-1.2/Setup.hs
-rw-r--r-- 0/0            3017 2012-11-09 08:20 ReadArgs-1.2/ReadArgs.cabal
argus% tar tvf /usr/portage/distfiles/ReadArgs-1.2.tar.gz 
drwxr-xr-x 0/0               0 2012-09-15 22:28 ReadArgs-1.2/
-rw-r--r-- 0/0            1533 2012-09-15 22:28 ReadArgs-1.2/LICENSE
-rw-r--r-- 0/0            3017 2012-09-15 22:28 ReadArgs-1.2/ReadArgs.cabal
-rw-r--r-- 0/0           10206 2012-09-15 22:28 ReadArgs-1.2/ReadArgs.hs
-rw-r--r-- 0/0             542 2012-09-15 22:28 ReadArgs-1.2/ReadArgsEx.hs
-rw-r--r-- 0/0              46 2012-09-15 22:28 ReadArgs-1.2/Setup.hs
argus% 

Thanks!

rampion commented 11 years ago

Not sure why it wasn't there before. It's present now in 1.2.1.