On Thu, Apr 15, 2010 at 1:20 AM, Volker Lendecke
<volker.lende...@sernet.de> wrote:
> On Wed, Apr 14, 2010 at 06:48:09PM -0400, Nico Kadel-Garcia wrote:

>> Can you publish the actual SRPM built for RHEL 5, or explain how you
>> were able to compile this package for RHEL 5?
> All I can say at this moment is that according to our
> internal procedures, the source RPM I referred you to is the
> one we are building from. I will investigate if something
> went wrong in posting this to the ftp server. We will
> certainly do everything necessary to ship the source code
> for the binaries you can download from our website to comply
> with the GPL.
> Can you please post the specific error message you are
> getting? It might be that our BuildRequires: line in the
> specfile is not correct or something else in our build
> environment is different than your host.
> Thanks,
> Volker

Thanks. I responded to Bjorn about the provenance of your SRPM.

Here are my error messages: This occurs on RHEL 5.4 and CentOS 5.4,
x86_64 architecture, in a normal home directory build and under mock.

$ rpmbuild --rebuild samba3-3.5.2-43.suse91.src.rpm
Installing samba3-3.5.2-43.suse91.src.rpm
warning: InstallSourcePackage: Header V4 DSA signature: NOKEY, key ID f4428b1a
error: Package already exists: %package debuginfo

Looking more deeply, I see your fascinating logic around
'this_is_redhat'., which seems to be failing. I've no idea why it's
failing, but I've previously found group statements to be a bit subtle
and unreliable unless you're very careful to set 'LANG' to something
predictable. In particular, in RHEL 5 with the default en_US.UTF8
setup, grep is case insensitive and should therefore be avoided unless
you set 'LANG=POSIX' or something reliable, first.

I'd like to suggest some cleaner logic for RedHat detection, as
included in this patch:

$ diff -u samba-3.5.spec.default  samba-3.5.spec
--- samba-3.5.spec.default      2010-04-06 06:14:28.000000000 -0400
+++ samba-3.5.spec      2010-04-15 07:10:57.000000000 -0400
@@ -7,7 +7,7 @@
 Url:           http://www.samba.org
 Vendor:                Service Network GmbH, Goettingen
 %define                releasefile     %(ls /etc/SuSE-release
/etc/UnitedLinux-release /etc/sles-release /etc/redhat-release
2>/dev/null | head -n1)
-%define                this_is_redhat  %(echo %{releasefile} | grep
-qi red && echo 1 || echo 0)
+%define                this_is_redhat  %([ -e /etc/redhat-release ]
&& echo 1 || echo 0)
 Distribution:  %(head -n1 < %{releasefile} || echo some SUSE-flavour ... )
 %if %(echo "%{distribution}" | grep -q "UnitedLinux" && echo 1 || echo 0 )
 %define                suse_ver        81

This patch allows compilation. I'm running a complete build right now.
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/options/samba

Reply via email to