Hello Josh and all, I and my team support this, as we found it very confusing getting started with Dyninst that the test suite was a separate entity. I do not know the history of why they are not merged, but as a user it would save time to have the option to compile the test suite while building Dyninst rather than conducting a second build step. I think many users welcome test suites as the first step in verifying a tool; and, in my opinion, it should be as easy as possible for users to bring up test suites.
Regards, Rashawn Knapp Software Development Engineer, Intel Corporation Platform Architecture & Performance Pole: JF2-1-M4 | MS: JF2-1-102-79| O: 503-264-4221 -----Original Message----- From: Dyninst-api [mailto:[email protected]] On Behalf Of Josh Stone Sent: Friday, May 06, 2016 12:21 PM To: [email protected] Subject: [DynInst_API:] RFC: unify the dyninst and testsuite repos How would folks feel about pulling the dyninst testsuite back into the main dyninst repository? I mentioned in a conversation about CI on github[1]. Having them in separate repos makes it harder to configure a lot of those CI tools. Any breaking API changes have to be pushed separately, inevitably causing a failed build in the interim. [1] https://github.com/dyninst/dyninst/pull/22 I think it would be much nicer for these to evolve in lockstep. It should be very easy to import it back with git-subtree, preserving all commit history. In fact, since it all used to be one repo, a lot of the git objects will even still be identical, so the metadata won't even grow much to bring it back. For the start, we can still keep them as independent CMake projects, with the testsuite requiring an installed dyninst root. Eventually, I think even this should be merged though, as I don't see a lot of use for the testsuite to act independently. It will also increase the source release tarball size from git-archive, but I think this isn't a big deal. DyninstAPI-9.1.0.tgz was ~8.7MB, and Testsuite-9.1.0.tgz was ~1.8MB, a relatively small addition. Both could be made smaller with bzip2 or even xz, if anyone cares. _______________________________________________ Dyninst-api mailing list [email protected] https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api _______________________________________________ Dyninst-api mailing list [email protected] https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api
