Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 338802 - [dev-lang/perl] Feature Request: make keepfile for vendor_perl
Summary: [dev-lang/perl] Feature Request: make keepfile for vendor_perl
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-26 14:00 UTC by Kent Fredric (IRC: kent\n) (RETIRED)
Modified: 2010-10-31 08:08 UTC (History)
1 user (show)

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


Attachments
Generating ConfigLocal.pm (g62DAmvc.txt,5.92 KB, text/plain)
2010-09-26 14:03 UTC, Kent Fredric (IRC: kent\n) (RETIRED)
Details
Message transcript. (transcript.txt,2.54 KB, text/plain)
2010-09-26 14:04 UTC, Kent Fredric (IRC: kent\n) (RETIRED)
Details
Proposed patch (patch.txt,534 bytes, patch)
2010-09-26 14:06 UTC, Kent Fredric (IRC: kent\n) (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-09-26 14:00:40 UTC
This is just a nicety. But it would 

a) Reduce the amount of errors spat out by the post-install configuration 
b) Reduce user confusion as to why that directory isn't there  till they install a module
c) make perl itself turn up with qfile $vendor_perl_dir 

The patch for this is reasonably straight-forward:

   keepdir "$VENDOR_ARCH"

anywhere in src_install.
Comment 1 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-09-26 14:03:10 UTC
Created attachment 248677 [details]
Generating ConfigLocal.pm

Here is a sample of the generation o ConfigLocal.pm , which while it says "Ignore any error" that part might scroll past the users view and they see only the errors.   We can nuke most of those errors, so why not?

Provided by Ch00k on #gentoo-perl
Comment 2 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-09-26 14:04:54 UTC
Created attachment 248678 [details]
Message transcript.
Comment 3 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-09-26 14:06:41 UTC
Created attachment 248680 [details, diff]
Proposed patch

LC_ALL=C TZ=GMT git diff perl-5.12.2-r1b.ebuild perl-5.12.2-r1.ebuild  > /tmp/patch.txt

Tested myself, installs ok. 

qlist dev-lang/perl | grep -i vendor 
/usr/lib64/perl5/vendor_perl/5.12.2/x86_64-linux-thread-multi/.keep_dev-lang_perl-0
Comment 4 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2010-09-26 14:24:56 UTC
Why did you assign bug-wranglers and just CC perl?
Either do straight to perl, or only bug-wranglers please.
Comment 5 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-09-26 14:59:28 UTC
(In reply to comment #4)
> Why did you assign bug-wranglers and just CC perl?
> Either do straight to perl, or only bug-wranglers please.
> 

Sorry Robin, I wasn't aware I had assigning power. If I could, I would, but I don't have the right bits to set assignments. ( Strangely, I can CC who I like ).

If you wish I not CC perl@ then fine, I'll not do that, but the power to assign is presently beyond me.
Comment 6 Torsten Veller (RETIRED) gentoo-dev 2010-09-27 04:07:09 UTC
http://git.debian.org/?p=perl/perl-5.12.git;a=blob;h=227dc105;hb=9d9ed099

Subject: Tweak enc2xs to follow symlinks and ignore missing @INC directories.
Bug-Debian: http://bugs.debian.org/290336

- ignore missing directories,
- follow symlinks (/usr/share/perl/5.8 -> 5.8.4).
Comment 7 Torsten Veller (RETIRED) gentoo-dev 2010-10-07 09:36:04 UTC
It's keepdir for now.
Comment 8 Torsten Veller (RETIRED) gentoo-dev 2010-10-31 08:08:39 UTC
Changed in dev-lang/perl-5.12.2-r2 and >=perl-core/Encode-2.40-r1 to ignore missing directories in INC as /etc/perl and /usr/local/lib/site_perl were added to INC too.