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.
> 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.