Re: [Setup-tool-hackers] No-brainer patch to make time-admin notcomplain on gentoo



On Sat, 2002-09-14 at 01:03, Markus Bertheau wrote:
> +sub check_gentoo
> +{
> +  open PORTAGE, "$xst_prefix/usr/portage" or return -1;
> +  return "gentoo";
> +}


This does not work. Say I use '/home/tambet/gnome2/' as
my $xst_prefix, there are no platform specific files, just stuff I
installed manually.

Also, this patch does not consider version of gentoo, is there just one
version of it?

Also, after opening a file, closing it would be nice also. Why not just
stat the file if you don't care about it's content?

In other words:

sub check_gentoo
{
	return "gentoo" if stat ("/usr/portage");
}

Tambet


_______________________________________________
setup-tool-hackers maillist  -  setup-tool-hackers@ximian.com
http://lists.ximian.com/mailman/listinfo/setup-tool-hackers



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]