public inbox for elfutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] Do not install profile.d/debuginfod.*sh files unless debuginfod is enabled
@ 2020-10-31 18:55 Dmitry V. Levin
  2020-10-31 20:34 ` Mark Wielaard
  0 siblings, 1 reply; 2+ messages in thread
From: Dmitry V. Levin @ 2020-10-31 18:55 UTC (permalink / raw)
  To: elfutils-devel

Fixes: 94708e964601 ("PR25461: add /etc/profile.d files for default $DEBUGINFOD_URLS")
Fixes: fee123fb771f ("PR25461: uninstall /etc/profile.d files for distuninstallcheck")
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
---
 config/ChangeLog   | 5 +++++
 config/Makefile.am | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/config/ChangeLog b/config/ChangeLog
index d3ecd6c6..c532f7e6 100644
--- a/config/ChangeLog
+++ b/config/ChangeLog
@@ -1,3 +1,8 @@
+2020-10-31  Dmitry V. Levin  <ldv@altlinux.org>
+
+	* Makefile.am (install-data-local, uninstall-local): Conditionalize
+	on DEBUGINFOD.
+
 2020-10-30  Mark Wielaard  <mark@klomp.org>
 
 	* elfutils.spec.in: Update for 0.182.
diff --git a/config/Makefile.am b/config/Makefile.am
index e56f32a5..ef28dab6 100644
--- a/config/Makefile.am
+++ b/config/Makefile.am
@@ -36,7 +36,6 @@ pkgconfigdir = $(libdir)/pkgconfig
 pkgconfig_DATA = libelf.pc libdw.pc
 if DEBUGINFOD
 pkgconfig_DATA += libdebuginfod.pc
-endif
 
 install-data-local:
 	$(INSTALL_DATA) profile.sh -D $(DESTDIR)$(sysconfdir)/profile.d/debuginfod.sh
@@ -44,6 +43,7 @@ install-data-local:
 
 uninstall-local:
 	rm -f $(DESTDIR)$(sysconfdir)/profile.d/debuginfod.sh $(DESTDIR)$(sysconfdir)/profile.d/debuginfod.csh
+endif
 
 if MAINTAINER_MODE
 $(srcdir)/elfutils.spec.in: $(top_srcdir)/NEWS

-- 
ldv

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Do not install profile.d/debuginfod.*sh files unless debuginfod is enabled
  2020-10-31 18:55 [PATCH] Do not install profile.d/debuginfod.*sh files unless debuginfod is enabled Dmitry V. Levin
@ 2020-10-31 20:34 ` Mark Wielaard
  0 siblings, 0 replies; 2+ messages in thread
From: Mark Wielaard @ 2020-10-31 20:34 UTC (permalink / raw)
  To: Dmitry V. Levin; +Cc: elfutils-devel

On Sat, Oct 31, 2020 at 09:55:28PM +0300, Dmitry V. Levin wrote:
> Fixes: 94708e964601 ("PR25461: add /etc/profile.d files for default $DEBUGINFOD_URLS")
> Fixes: fee123fb771f ("PR25461: uninstall /etc/profile.d files for distuninstallcheck")
> Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
> [...]
> @@ -1,3 +1,8 @@
> +2020-10-31  Dmitry V. Levin  <ldv@altlinux.org>
> +
> +	* Makefile.am (install-data-local, uninstall-local): Conditionalize
> +	on DEBUGINFOD.

Makes sense, if we don't install libdebuginfo the profile wouldn't do
anything. Pushed.

Thanks,

Mark

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-10-31 20:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-31 18:55 [PATCH] Do not install profile.d/debuginfod.*sh files unless debuginfod is enabled Dmitry V. Levin
2020-10-31 20:34 ` Mark Wielaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).