[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