Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

files from tests/ not included in tarball #13

Open
danielj7 opened this Issue Sep 17, 2010 · 4 comments

Comments

Projects
None yet
2 participants

The files from tests/ need to be added to EXTRA_DIST in tests/Makefile.am as they don't get added to the tarball which breaks the test suite.

Owner

petdance commented Sep 17, 2010

patch please! Pleeeeease!

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 3da6639..65aec0f 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -1,3 +1,20 @@
 # Nothing
 
-EXTRA_DIST = input/ output/ accessibility/
+EXTRA_DIST = \
+       input/ \
+       output/ \
+       accessibility/ \
+       accesscases.txt \
+       acctest.cmd \
+       alltest.cmd \
+       alltest1.cmd \
+       onetest.cmd \
+       onetesta.cmd \
+       testaccess.sh \
+       testaccessone.sh \
+       testall.sh \
+       testcases.txt \
+       testone.sh \
+       testxml.sh \
+       xmlcases.txt \
+       xmltest.cmd
Owner

petdance commented Sep 17, 2010

I'll apply this later tonight.

The slicker, githubbier way to do this (and not mess with markdown in the ticket) is to fork my petdance/tidyp repo, check out your forked version, and make your changes there, and push back to github. At that point, your change will be on danielj7/tidyp. Then, you send me a pull request and I can pull your change directly out of the repo. No patch sending necssary. It's super slick.

http://help.github.com/forking/

http://help.github.com/pull-requests/

Thanks! I'm still learning my way around git. Too many years of using cvs has caused brain damage. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment