Re: [Cppunit-devel] Patch for VC++ testrunner...

[Bastiaan B. <bas...@li...>]

Baptiste Lepilleur wrote:

>     I finally made the patch. It's in the attached file. The
> bin-examples.tar.gz contains the binary files and should be unpacked above
> examples/.
>

I see.
One question: is the test runner MSVC++ specific per definition? If not, I'd prefer to put the
files in a win32 sub directory, rather than 'msvc6'. People may want to compile under a different
compiler than MSVC++.

>
>     The patch includes Michael's Feather test runner (slightly modified to
> avoid conflict when compiling). I'll include my enhancement later, once I
> cleaned it a bit. It's also include the original sample application that
> show how to use the test runner. CppUnit extensions have also been added
> (include/cppunit/extensions).
>
>     I also added the makefile.am in each new directory (hope I did not
> forget one), and added them to the SUBDIR variable. The dsp and dsw have
> been added to the EXTRA_DIST. Should the .cpp and .h be added to ? If so,
> could this be done using shell command (something akin to EXTRA_DIST = `ls
> *.h *.cpp *.ico *.rc *.bmp`) ?
>

I'm not sure if a shell command would work. I'll try to find out.

>
>     I played around with cvs. How to you get it to make a patch containing
> the binary file ? This is the command line I'd used (I didn't used it
> because of the EOL problem with the dsp/dsw, but once it is fixed I plan to
> use that):
>
> cvs rdiff -R -u -r tg-cppunitsf-sf-1 -r tg-cppunitsf-trunc-patch-testrunner
> sf/cppunit > testrunner.patch
>
>     How do you get it to do: "for the binary file, I want you to include the
> version of the second revision if they are different." Same question using
> diff.
>

According to the man/info pages, diff cannot output the contents of binary files.
But again, I'll see if some solution is available.

>
>     Once all this is merged, could a tarball be generated so I can check if
> no file is missing ? Once this is validated, we could probably make a
> release. I think I'll write a small html page "build guide for win32".
>

Once your patches have been merged, I'll put a tar ball in the projects FTP section.
Could you send me the html page? Seems like a good addition in the doc dir.


Bastiaan


>
>     Baptiste.
> ---
> Baptiste Lepilleur <gai...@fr...>   http://gaiacrtn.free.fr/index.html
> Author of The Text Reformatter, a tool for fanfiction readers and writers.
> Language: English, French (Well, I'm French).
>
>   ------------------------------------------------------------------------
>                                                     Name: cppunit-testrunner-bin-examples.tar.gz
>    cppunit-testrunner-bin-examples.tar.gz           Type: Unix Tape Archive (application/x-tar)
>                                                 Encoding: base64
>                                          Download Status: Not downloaded with message
>
>                                          Name: cppunit-testrunner.patch.gz
>    cppunit-testrunner.patch.gz           Type: application/x-compressed
>                                      Encoding: base64
>                               Download Status: Not downloaded with message