On 2009-02-26 14:24:02 +0100, Michael Schroeder wrote:
On Thu, Feb 26, 2009 at 01:16:49PM +0100, Marcus Hüwe wrote:
These are valid package names but aren't parsed correctly by Build::readdeps() which is due to ... if ($s =~ s/^P:(.*):$/$1/) { my $pkgid = $s; $s =~ s/-[^-]+-[^-]+-[^-]+$//; $provides{$s} = \@ss; ... # the same for provides
Yep, Build::readdeps() is broken. It still is only able to parse the old autobuild format while the Build.pm code nowadays writes some slight different format. Luckily this part of readdeps() isn't used at all, thus we're free to fix it.
Ok this sounds good. Yet another question: is the current format used somewhere else? Because otherwise it would be cool if it's possible to slightly change the format. At the moment it has the following structure: ... I:<pkgid>: <name>-<version>-<release> <buildtime> ... My proposal is to separate <name>, <version> and <release> with a whitespace instead of the dash because it would be easier to parse the name version and release of a package (I need this info in order to send back a "clean" buildinfo). Marcus -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org