All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miloslav Trmac <mitr@redhat.com>
To: klausk@br.ibm.com
Cc: "Linux-audit@redhat.com" <Linux-audit@redhat.com>
Subject: Re: building audit-1.6.x is broken in RHEL5[.x]
Date: Thu, 11 Oct 2007 04:49:13 +0200	[thread overview]
Message-ID: <470D8F29.4080409@redhat.com> (raw)
In-Reply-To: <1191942839.31507.20.camel@klausk.br.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 2411 bytes --]

Hello,
Klaus Heinrich Kiwi napsal(a):
>   I'm unable to build audit 1.6.x in RHEL5 or RHEL5 update1 (as of
> snap8) using rpmbuild. I'm directing you this report as looks like the
> problem lies in the system-config-audit package.
The correct solution is to:
* create the audit tarball using (make dist)
* remove the autotools invocations from the spec file.

This should make the build process independent of versions - or even
presence - of autotools on the build machine.  OTOH patching the build
machinery in the spec file might become more difficult.


Now, for the problems encountered while using the current tarballs:
> Seems like something is failing to replace @DATADIRNAME@ for 'share'
> while intltoolize'ing, even though there's a m4 macro within
> m4/gettext.m4 that says:
>     dnl For backward compatibility. Some Makefiles may be using this.
>     DATADIRNAME=share
>     AC_SUBST(DATADIRNAME)
> 
> I don't understand why the AC_SUBST(DATADIRNAME) macro isn't working and
> I'd like to build the package in an unmodified RHEL5 (GA) server.
The AC_SUBST(DATADIRNAME) is performed only in a branch that isn't
executed in system-config-audit (because AM_GNU_GETTEXT has the
"external" parameter).  intltool relies on gettext providing the
substitution, and gettext stopped doing so.

> Anyone
> have a clue on how to fix this? Isn't it worth fixing it upstream?
The correct fix is to modify intltool not to rely on the substitution;
this is already fixed in later intltool versions.


> This time, seems like system-config-audit.desktop got install'ed to a wrong location:
> test -z "/applications" || mkdir -p -- "/var/tmp/audit-1.6.2-root/applications"
>  /usr/bin/install -c -m 644 'system-config-audit.desktop' '/var/tmp/audit-1.6.2-root/applications/system-config-audit.desktop'
> 
> In the generated Makefile, I see:
> applicationsdir = $(datarootdir)/applications
> but no other references to datarootdir
datarootdir is only provided by autoconf >= 2.59c; the autoconf in RHEL5
is too old.  This problem happens only because configure is regenerated
during the build process.


As a workaround, the attached spec file change works for me when
building rawhide audit-1.6.2-3 on RHEL5.


> btw, why is there a 'Makefile.in.in~' file inside
> system-config-audit/po?
Because the audit distribution tarball is apparently created manually,
and the file was accidentally packaged as well.
	Mirek

[-- Attachment #2: audit.hack.patch --]
[-- Type: text/x-patch, Size: 642 bytes --]

--- audit.spec.hack	2007-10-11 03:41:04.000000000 +0200
+++ audit.spec	2007-10-11 04:02:01.000000000 +0200
@@ -83,8 +83,8 @@
 mkdir -p $RPM_BUILD_ROOT/%{_lib}
 mkdir -p $RPM_BUILD_ROOT/%{_libdir}/audit
 mkdir -p $RPM_BUILD_ROOT/%{_var}/log/audit
-make DESTDIR=$RPM_BUILD_ROOT install
-make -C system-config-audit DESTDIR=$RPM_BUILD_ROOT install-fedora
+make DESTDIR=$RPM_BUILD_ROOT datarootdir=%{_datadir} DATADIRNAME=share install
+make -C system-config-audit DESTDIR=$RPM_BUILD_ROOT datarootdir=%{_datadir} DATADIRNAME=share install-fedora
 
 mkdir -p $RPM_BUILD_ROOT/%{_libdir}
 # This winds up in the wrong place when libtool is involved

[-- Attachment #3: Type: text/plain, Size: 0 bytes --]



      parent reply	other threads:[~2007-10-11  2:49 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-09 15:13 building audit-1.6.x is broken in RHEL5[.x] Klaus Heinrich Kiwi
2007-10-09 15:47 ` klausk
2007-10-09 16:01   ` Steve Grubb
2007-10-09 19:21     ` klausk
2007-10-11  2:49 ` Miloslav Trmac [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=470D8F29.4080409@redhat.com \
    --to=mitr@redhat.com \
    --cc=Linux-audit@redhat.com \
    --cc=klausk@br.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.