Martin, I make a patch to distinguish RHEL and Oracle EL at Distro.ycp. Can you view it? Thanks Pearly On 08/14/2008 01:20 PM, Elena Zannoni wrote:
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