[Bug 304316] New: hal: use better libexecdir
https://bugzilla.novell.com/show_bug.cgi?id=304316 Summary: hal: use better libexecdir Product: openSUSE 10.3 Version: Beta 2 Platform: Other OS/Version: Other Status: NEW Severity: Enhancement Priority: P5 - None Component: Basesystem AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: sbrabec@novell.com QAContact: qa@suse.de Found By: --- According to packaging conventions, libexecdir should be based on %{_prefix}/lib, not %{_libdir}. See also: bug 157894 This change will allow to use 32-bit addons on 64-bit systems, which is IMHO possible, because addons are binaries. Please note, that this change must be done in all addon providing packages. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 Matej Horvath <mhorvath@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |dkukawka@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316#c1 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |sbrabec@novell.com --- Comment #1 from Danny Kukawka <dkukawka@novell.com> 2007-08-24 11:32:24 MST --- We use this in configure: %configure \ --libexecdir=%{_libexecdir}/hal \ if _libexecdir is on 64bit /usr/lib64 someone should may fix _libexecdir or not? Not sure if this is really a HAL problem -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316#c2 Stanislav Brabec <sbrabec@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |157894 Status|NEEDINFO |NEW Info Provider|sbrabec@novell.com | --- Comment #2 from Stanislav Brabec <sbrabec@novell.com> 2007-09-03 06:28:19 MST --- Bug 157894 was cosidered as INVALID. So using the default incorrect %{_libexecdir} should be considered as deprecated. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User dkukawka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c3 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |sbrabec@novell.com --- Comment #3 from Danny Kukawka <dkukawka@novell.com> 2008-04-24 05:31:32 MST --- 1) why is there no warning about use %{_libexecdir} in our build system if it is deprecated? 2) why should someone try to mix a 64bit HAL package with (external, not with hal.rpm shipped addons) 32bit addons? The addons need libhal and this is already 64bit on a 64bit system. I don't see atm the advantage of changing --libexecdir -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c4 Stanislav Brabec <sbrabec@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|sbrabec@novell.com | --- Comment #4 from Stanislav Brabec <sbrabec@novell.com> 2008-04-24 08:13:44 MST --- 1. See bug 157894. %{_libexecdir} is not deprecated, only the default value is bad. Other distros have bad value as well, we are bug-compatible. But we are bi-arch, so our packages may break. 2. Well, hal is definitely not affected by libexecdir badness: grep /usr/lib64/hal $(rpm -ql hal) Binary file /usr/sbin/hald matches No library is affected, only hald matches. So yes, just now there is no advantage and no disadvantage. I reported this bug against all packages using bad inside %{_libdir}. Real brokenness appears, only: 1. libexecdir is inside /usr/lib64 and 2. grep libexecdir_of_the_package $(rpm -ql package_name) returns any match in shared library on 64-bit system and 3. we provide -32bit package with this library and 4. we do not provide libexecdir helpers in -32bit package (not providing is a default behavior) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User dkukawka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c5 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |WONTFIX --- Comment #5 from Danny Kukawka <dkukawka@novell.com> 2008-04-24 09:43:38 MST --- Thanks for the information. If hal is not addected by this problem and if there are no advantages in changing the current hal.spec, I close the bug for now as WONTFIX. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c6 Stanislav Brabec <sbrabec@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Component|Basesystem |Basesystem Product|openSUSE 10.3 |openSUSE 11.0 Resolution|WONTFIX | Version|Beta 2 |Factory --- Comment #6 from Stanislav Brabec <sbrabec@novell.com> 2008-04-30 07:17:40 MST --- Reopening. I did not notice another way, how it could break: Third party supplier provides hal addon, which is 32-bit only. I think that it could work on 64-bit system without any problem, but it will not. Hald daemon will read FDI file and search for addon in /usr/lib64/hal. But the addon will be installed to /usr/lib/hal and will not be found. It would be nice to fix it for 11.1 and SLES11. There are upstream problems related: hald_runner correctly uses PACKAGE_LIBEXEC_DIR, but PACKAGE_SCRIPT_DIR is (very probably incorrectly) derived from libdir. Reporting as https://bugs.freedesktop.org/show_bug.cgi?id=15767 This upsteam problem has to be fixed first. And yet another related (upstream) problem: hal.pc provides no way to detect location of hald addon directory. Reporting as https://bugs.freedesktop.org/show_bug.cgi?id=15768 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User dkukawka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c7 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEEDINFO Info Provider| |sbrabec@novell.com --- Comment #7 from Danny Kukawka <dkukawka@novell.com> 2008-04-30 08:41:22 MST --- How can you run 32bit addon against a 64bit libhal (which is needed by all HAL addons) and a 64bit HAL? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c8 Stanislav Brabec <sbrabec@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |REOPENED Info Provider|sbrabec@novell.com | --- Comment #8 from Stanislav Brabec <sbrabec@novell.com> 2008-05-02 04:21:10 MST --- 32bit addon will link against hal-32bit, which is part of our distribution. Addons are binaries, so architecture does not need to match architecture of the caller (even if starting from library, but hal addons are started by hald). I did not tested bi-arch addon, but I see no reason, why it will not work (except this bug). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User dkukawka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c9 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEEDINFO Info Provider| |sbrabec@novell.com --- Comment #9 from Danny Kukawka <dkukawka@novell.com> 2008-05-07 06:45:39 MST --- There are now comments on the bugs you filed on fd.o. Could you please take a look at it and comment? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c10 --- Comment #10 from Stanislav Brabec <sbrabec@novell.com> 2008-05-09 04:36:56 MST --- Yes, I will do it and provide comments on the consensus. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|Basesystem |Basesystem Product|openSUSE 11.0 |openSUSE 11.1 Version|Factory |Alpha 0 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User behlert@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c11 --- Comment #11 from Stefan Behlert <behlert@novell.com> 2008-07-14 08:34:59 MDT --- Stanislav, any update here? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c12 Stanislav Brabec <sbrabec@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |REOPENED Info Provider|sbrabec@novell.com | --- Comment #12 from Stanislav Brabec <sbrabec@novell.com> 2008-09-22 09:13:44 MDT --- I have just provided a new patches to the upstream bugzilla. They are safe to apply, but you have to decide about new defaults: - Use --libexecdir=%{_libdir}/hal: keep compatibility, keep bi-arch problems (32-bit scripts can't work in 64-bit distributions) - Use --libexecdir=%{_prefix}/lib/hal: break compatibility, but fix bi-arch problems (32-bit scripts can work in 64-bit distributions) If you decide the second, you can create compatibility symlink. Due to RPM brokenness it is a bit complicated, but possible e. g. in %post (untested): %post %if %{_lib} != lib mv 2>/dev/null usr/lib64/hal/* . || : rmdir 2>/dev/null usr/lib64/hal || : mkdir -p usr/lib64 ln -s ../lib/hal usr/lib64/hal %endif -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 Danny Kukawka <dkukawka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - None |P4 - Low -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=304316 User sbrabec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=304316#c13 --- Comment #13 from Stanislav Brabec <sbrabec@novell.com> 2008-10-15 08:44:00 MDT --- Patch is already provided and waiting for review: https://bugs.freedesktop.org/show_bug.cgi?id=15768#c4 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com