Sorry, there's more stuff that's broken. Turns out the SuSE.pm module is
crappy, and that one served as basis for Bernard's version of SLES.pm.
I'll fix that in trunk asap. Main issue:
if (-f "/etc/$detect_package") {
$release_string = `cat /etc/$detect_package`;
won't deal with chroot directories... and the $detect_package is in this case
a file and not sles-release.
Regards,
Erich
On Tuesday 10 October 2006 15:19, Erich Focht wrote:
> Hi Chris,
>
> looking into your SLES.pm I see two issues:
>
>
> > my $detect_package = "SuSE-release";
> This should be "sles-release". It is the package name which contains the
> release information.
>
>
> >
> > if ($release_string =~ SUSE Linux Enterprise Server 10 (\d+.\d+)/) {
> >
> > my $os_version = $1;
>
> For the version you actually want to match for the "10", so the regexp should
> be:
>
> > if ($release_string =~ m/SUSE Linux Enterprise Server (\d+) \(.*\)/) {
>
> Will check if this is in the trunk, if yes, I'll fix it. But maybe you could
> try it.
>
> BTW: the output of "$OSCAR_HOME/scripts/distro-query -v" should give you the
> result of the OS detection, too. This is more practical for testing it than
> the full install...
>
> Regards,
> Erich
>
>
>
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Oscar-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/oscar-devel
