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@oracle.com -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org For additional commands, e-mail: yast-devel+help@opensuse.org