[mythtv] Ticket #9338: Sandbox violation (gentoo)
Marc Tousignant
myrdhn at gmail.com
Mon Dec 6 10:27:38 UTC 2010
-----Original Message-----
From: MythTV [mailto:noreply at mythtv.org]
Sent: Sunday, December 05, 2010 9:55 PM
Cc: mythtv-commits at mythtv.org
Subject: Re: Ticket #9338: Sandbox violation (gentoo)
#9338: Sandbox violation (gentoo)
--------------------------------------+----------------------------
Reporter: MarcT <myrdhn@…> | Owner: beirdo
Type: Patch - Bug Fix | Status: closed
Priority: minor | Milestone: unknown
Component: Plugin - MythWeather | Version: Trunk Head
Severity: medium | Resolution: Won't Fix
Keywords: Sandbox violation | Ticket locked: 0
--------------------------------------+----------------------------
Changes (by beirdo):
* status: new => closed
* resolution: => Won't Fix
Comment:
You will have to find another way to fix your packaging issue. The use of
qmake for installing these scripts was removed because it doesn't work
100% right, and I'm not putting it back to using qmake. Find a way that
involves keeping the new Makefiles, not backtracking.
--
Beirdo,
I solved this differently.
Gentoo ebuilds install into a interim location before actually installing the files into the system. This allows us to make sure there are not going to be file collisions from different packages. The Makefile.inc in the scripts folder bypasses this attempting to write directly to a system folder because it was not generated by qmake, like you said.
The below fixes this without reverting back to qmake. Not sure if it would be worthwhile for it to be patched into the source or if we file this as another Gentoo-ism.
MarcT
-- mythtv.orig/mythplugins/mythweather/mythweather/scripts/Makefile.inc 2010-12-05 17:48:10.000000000 -0500
+++ mythtv/mythplugins/mythweather/mythweather/scripts/Makefile.inc 2010-12-06 04:59:49.000000000 -0500
@@ -24,13 +24,13 @@
install-scripts:
-[ "${SCRIPTS}" != "" ] && \
- ${INSTALL} -m 755 -d ${INSTDIR}/ && \
- for i in ${SCRIPTS} ; do ${INSTALL} -m 755 $$i ${INSTDIR}/ ; done
+ ${INSTALL} -m 755 -d $(INSTALL_ROOT)${INSTDIR}/ && \
+ for i in ${SCRIPTS} ; do ${INSTALL} -m 755 $$i $(INSTALL_ROOT)${INSTDIR}/ ; done
install-data:
-[ "${DATA}" != "" ] && \
- ${INSTALL} -m 755 -d ${INSTDIR}/ && \
- for i in ${DATA} ; do ${INSTALL} -m 644 $$i ${INSTDIR}/ ; done
+ ${INSTALL} -m 755 -d $(INSTALL_ROOT)${INSTDIR}/ && \
+ for i in ${DATA} ; do ${INSTALL} -m 644 $$i $(INSTALL_ROOT)${INSTDIR}/ ; done
uninstall-scripts:
-[ "${SCRIPTS}" != "" ] && \
More information about the mythtv-dev
mailing list