Bug 214273

Summary: inst-source-utils perl module in subdir of /usr/bin is questionable/ugly
Product: [openSUSE] openSUSE 10.3 Reporter: Andreas Hanke <andreas.hanke>
Component: YaST2Assignee: Lars Vogdt <lars.vogdt>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Minor    
Priority: P5 - None    
Version: Alpha 1   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Patch for create_package_descr
Patch for the spec file

Description Andreas Hanke 2006-10-22 15:18:06 UTC 
inst-source-utils creates a subdirectory in /usr/bin and places a perl module there.

Is that necessary? At least in a default installation, no other package creates a subdirectory in /usr/bin, and /usr/bin is intended for binaries.

Maybe the package can place its perl module into an otherwise equally named subdirectory of /usr/lib or /usr/share instead of /usr/bin.

(it seems that its predecessor from autoyast2-utils had this module embedded into the script)
Comment 1 Lars Vogdt 2006-10-25 10:17:27 UTC 
Yes. I know that this directory is an ugly hack at the moment.

Including the functions in the create_package_descr script is no way to go (we use the functions of RPMQ.pm in other scripts, too, because they are much faster than the normal rpm funtions).

So I'll see if I can create a regular CPAN Module - and package the RPMQ.pm in the "normal" CPAN module way. But this can take some time (it's my first CPAN module ... ;-)
Comment 2 Andreas Hanke 2007-02-19 22:37:46 UTC 
It shouldn't be so hard to get this stuff out of /usr/bin. This does not require creating a CPAN module. That would be an unrelated enhancement.
Comment 3 Andreas Hanke 2007-02-19 22:38:31 UTC 
Created attachment 120000 [details]
Patch for create_package_descr
Comment 4 Andreas Hanke 2007-02-19 22:39:36 UTC 
Created attachment 120001 [details]
Patch for the spec file
Comment 5 Lars Vogdt 2007-02-26 13:30:39 UTC 
Boing! Sometimes the bug is prior to the monitor. Thanks for opening my eyes!

Fixed package submitted to stable. As this package is normally auto-generated, I did not inlcude your patches - but the result should be same.