Mailinglist Archive: yast-devel (73 mails)

< Previous Next >
Re: [yast-devel] YaST for (RH)EL5 patch - yast2-2.13.104
  • From: Elena Zannoni <elena.zannoni@xxxxxxxxxx>
  • Date: Thu, 14 Aug 2008 13:20:32 -0400
  • Message-id: <48A46960.9020904@xxxxxxxxxx>
Hi Martin, thanks for looking this over and starting the commits.
I was perusing the yast-commit list, and I noticed this bit in your
commit from the 12th of August:


+ * Is it Fedora based? RHEL, Oracle, ...
+ */
+global boolean fedora () {
+ if (_distro == nil)
+ {
+ if (SCR::Read (.target.size, "/etc/fedora-release") != -1)
+ {
+ _distro = "fedora";
+ y2milestone ("Found Fedora");
+ }
+ }
+
+ return _distro == "fedora";
+}



The comment is incorrect, there is no /etc/fedora-release file in OEL or RHEL,
they have
/etc/redhat-release instead (OEL has actually 2 files, /etc/redhat-release and
/etc/enterprise-release), so I'd say RHEL and OEL need a separate function?

thanks
elena


--
Elena Zannoni, Oracle
Senior Engineering Manager, Tools/Languages - Linux Engineering
Blog: http://blogs.oracle.com/ezannoni
Email: elena.zannoni@xxxxxxxxxx

--
To unsubscribe, e-mail: yast-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: yast-devel+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups
References