bugGNU Octave - Bugs: bug #36830, Create pkg prefix directory, if it...

 
 

bug #36830: Create pkg prefix directory, if it does not exist

Submitted by:  None
Submitted on:  Thu 12 Jul 2012 04:48:37 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedAssigned to: Jordi Gutiérrez Hermoso <jordigh>
Originator Name: Rafael LaboissiereOriginator Email: -unavailable-
Open/Closed: ClosedRelease: dev
Operating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Thu 12 Jul 2012 03:21:29 PM UTC, comment #1:

Thanks. I edited your patch's metadata a little (fixed your name and slightly reworded the commit message), and I pushed it, along with my own fix for checking if the prefix directory can be created:

http://hg.savannah.gnu.org/hgweb/octave/rev/7371e127c351
http://hg.savannah.gnu.org/hgweb/octave/rev/a82fcd612c8f

Jordi Gutiérrez Hermoso <jordigh>
Project AdministratorIn charge of this item.
Thu 12 Jul 2012 04:48:37 AM UTC, original submission:

The pkg.m script is smart enough to create the archprefix directory when it is not present in the system (the examples below are run in a Debian GNU/Linux system with Octave 3.6.2 installed):

This is not the case for the prefix directory:

The trivial patch attached to this bug report fixes this issue.

Note that this problem has been reported as Bug##681064 against the Debian package octave. In the discussion around that bug report in Debian, Jordi Gutiérrez suggested that the code has to be improved anyway ("What if creating the directory fails? Or do we know at that point that this is a writable location?"). I think I agree with him, but the changes need to be done to both the prefix- and the archprefix-related code in pkg.m

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #26182:  create-pkg-prefix-dir.diff added by None (897B - text/x-patch - Patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by None (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 12 Jul 2012 03:21:29 PM UTCjordighStatusNone=>Fixed
      Assigned toNone=>jordigh
      Open/ClosedOpen=>Closed
    Thu 12 Jul 2012 04:48:37 AM UTCNoneAttached File-=>Added create-pkg-prefix-dir.diff, #26182

    Back to the top


    Powered by Savane 3.1-cleanup1