Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 837536 - app-portage/g-cpan creates folder in /var/tmp which may be deleted any time
Summary: app-portage/g-cpan creates folder in /var/tmp which may be deleted any time
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-09 22:57 UTC by Jonas Stein
Modified: 2022-04-15 09:47 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonas Stein gentoo-dev 2022-04-09 22:57:29 UTC
`man hier`:

/var/tmp
              Like /tmp, this directory holds temporary files stored for an unspecified duration.

The ebuild creates a folder there. No body knows how long it remains there. This also triggers a correct warning by qcheck:


Checking app-portage/g-cpan ...
 AFK: /var/tmp/g-cpan


src_install() {
    perl-module_src_install
    if ! use prefix; then
        diropts -m0775 -o portage -g portage
    else
        diropts -m0775
    fi
    dodir "/var/tmp/g-cpan"
    dodir "/var/log/g-cpan"
    keepdir "/var/log/g-cpan"
}



Reproducible: Always
Comment 1 Larry the Git Cow gentoo-dev 2022-04-10 17:20:02 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage.git/commit/?id=8ded447a1d194ad0190a174f7bb330417685d861

commit 8ded447a1d194ad0190a174f7bb330417685d861
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-04-10 16:51:02 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-04-10 17:19:57 +0000

    install-qa-check.d/20runtime-directories: warn on /var/tmp too
    
    Bug: https://bugs.gentoo.org/493154
    Bug: https://bugs.gentoo.org/837536
    Signed-off-by: Sam James <sam@gentoo.org>

 bin/install-qa-check.d/20runtime-directories | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Comment 2 Larry the Git Cow gentoo-dev 2022-04-15 08:37:03 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bb044a3dd58f5d6ac0fa66dfe07daf3c6773480e

commit bb044a3dd58f5d6ac0fa66dfe07daf3c6773480e
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-04-15 08:35:57 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-04-15 08:36:54 +0000

    sys-apps/portage: backport new QA checks; /run warning removal
    
    portage.git is currently undergoing a fair bit of refactoring
    and other non-trivial changes, so let's backport some useful bits.
    
    - Backport QA checks (eend w/o ebegin, ebegin w/o eend, /var/tmp install dir)
    - Drop obsolete /run-not-mounted warning
    
    Bug: https://bugs.gentoo.org/835824
    Bug: https://bugs.gentoo.org/835823
    Bug: https://bugs.gentoo.org/832853
    Bug: https://bugs.gentoo.org/493154
    Bug: https://bugs.gentoo.org/837536
    Signed-off-by: Sam James <sam@gentoo.org>

 sys-apps/portage/files/3.0.30-qa-checks.patch      | 137 ++++++++++
 .../portage/files/3.0.30-revert-run-warn.patch     |  64 +++++
 sys-apps/portage/portage-3.0.30-r2.ebuild          | 276 +++++++++++++++++++++
 3 files changed, 477 insertions(+)