Re: pear convert: Invalid package.xml

Sebastian Bergmann wrote: > I am trying to convert PHPUnit2's package.xml to package2.xml: > > sb@wopr-mobile PHPUnit2 % pear convert > Invalid tag order in <dependencies><optional>, found <php> expected > one of "package, subpackage, extension" > PEAR_Packagefile_v2::toPackageFile: invalid package. > > This is the package.xml part it complains about: > > <deps> > <dep type="php" rel="ge" version="5.1.2" /> > <dep type="php" rel="ge" version="5.1.3" > optional="yes" /> > ... > </deps> > > Is the following package2.xml part (that I converted manually) legal? > > <dependencies> > <required> > <php> > <min>5.1.2</min> > </php> > ... > </required> > <optional> > <php> > <min>5.1.3</min> > </php> > ... > </optional> > </dependencies> This is indeed not legal, as this was a hack taking advantage of PEAR 1.3.x's not-so-brilliant dep calculations to display a notice to users. If you support 5.1.2 and later, use a 5.1.2 dep, and write "5.1.3 and higher is recommended" in the docs and in the package information/release notes. There is no such thing as an optional dependency on php, and the new package.xml format more accurately reflects dependencies. Sorry for the inconvenience. Thanks, Greg