Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 454874

Summary: dev-lang/tcc-0.9.26_pre1 - access violation in install tcc-doc.info /usr/share/info
Product: Gentoo Linux Reporter: Samuel Bauer <samuel.bauer>
Component: [OLD] DevelopmentAssignee: Maintainers for Miscelleneous Language Packages [OBSOLETE] <lang-misc+disabled>
Status: RESOLVED FIXED    
Severity: normal CC: dan, jason.mours, patrick
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: build.log
tcc-0.9.26_pre1.ebuild.patch

Description Samuel Bauer 2013-01-31 17:23:45 UTC
Created attachment 337468 [details]
build.log

Write deny
Comment 1 Samuel Bauer 2013-01-31 17:25:21 UTC
Created attachment 337470 [details, diff]
tcc-0.9.26_pre1.ebuild.patch

Sets DESTDIR="${D}" in make install to fix write deny
Comment 2 Alessandro Di Federico 2013-02-03 22:00:50 UTC
I can confirm it works.
Comment 3 Patrick Lauer gentoo-dev 2013-02-04 05:22:53 UTC
docdir="${D}"/usr/share/doc/${PF} \

that's already in the ebuild, and I cannot reproduce the failure.

... so what makes our systems different enough so that you can make it fail?
Comment 4 Samuel Bauer 2013-02-04 05:50:10 UTC
My gentoo install isn't modified in any manner for /usr/ directory.
It's quite old (~5years), but has been maintained up to date regulary since my last install.
Another user (comment #2), seems to encounter this problem.

I don't think one need to use ${D} for each install directory, at the risk to have a missing one, but rather to set it "globally" through DESTDIR="${D}". It the common way used by EAPI4 (I think it's same in EAPI5 used by this ebuild) in src_install.
Comment 5 Daniel M. Weeks 2013-03-22 20:01:05 UTC
(In reply to comment #3)
> docdir="${D}"/usr/share/doc/${PF} \
> 
> that's already in the ebuild, and I cannot reproduce the failure.
> 
> ... so what makes our systems different enough so that you can make it fail?

I just experienced this bug as well. The failure is because the tcc configure script only uses ${infodir} to create a canonical path for info files so

docdir="${D}"/usr/share/doc/${PF} \

has no effect.

I added

infodir="${D}"/usr/share/info/${PF} \

to the ebuild to correct the problem which I believe is "more correct" than specifying DESTDIR="${D}".
Comment 6 Jason Mours 2013-04-08 21:52:25 UTC
Is the patch going to be pushed to portage? I'm having tcc problems with my build server, and while I'm not apposed to using ~amd64 testing, I won't use a patch unless it's in portage on this machine.
Comment 7 Patrick Lauer gentoo-dev 2013-04-16 09:36:58 UTC
+  16 Apr 2013; Patrick Lauer <patrick@gentoo.org> tcc-0.9.26_pre1.ebuild:
+  Fix for sandbox violation #454874

Reproduced, and fixed