Comment 14 for bug 2015234

Revision history for this message
Andrew Johnson (anj) wrote :

> No problem found

What about testing the code sections that run for the other version control systems?

This script was designed to work with EPICS applications (not just Base itself) that are stored in Darcs, Mercurial, Git, Subversion or Bazaar, and each of those has a separate section of the script for obtaining version information from the VCS which could be affected by that change. The Darcs code at least appears to handle multi-line results. I'm not saying that there _is_ a problem, but this kind of change could very easily break one of the others unless you localize the $/ variable as I showed above. If you don't do that, they ought to be tested too.