tidyp icon indicating copy to clipboard operation
tidyp copied to clipboard

files from tests/ not included in tarball

Open danielj7 opened this issue 15 years ago • 4 comments

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.

danielj7 avatar Sep 17 '10 15:09 danielj7

patch please! Pleeeeease!

petdance avatar Sep 17 '10 15:09 petdance

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

danielj7 avatar Sep 17 '10 16:09 danielj7

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/

petdance avatar Sep 17 '10 17:09 petdance

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

danielj7 avatar Sep 17 '10 19:09 danielj7