bugGNU Octave - Bugs: bug #32839, pkg build/install does not work...

 
 

bug #32839: pkg build/install does not work with directories

Submitter:  Orion Poplawski <opoplawski>
Submitted:  Fri 18 Mar 2011 04:11:03 PM UTC
   
 
Category:  Octave Function Severity:  2 - Minor
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Wont Fix Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 25 Aug 2021 06:31:36 AM UTC, comment #9: 

CODE SPRINT: This item was open for a long time and no further progress is to be expected.  Thus it will be closed now.  If you care for this bug or you think it was a wrong decision, please write details for your argument and it can be reopened again.

Kai Torben Ohlhus <siko1056>
Group Member
Wed 07 Dec 2016 03:08:11 PM UTC, comment #8: 

With the update to 4.2, for Fedora I've dropped the pkg patch and modified the octave pkg build/install macros to repack the source if necessary.  So I'm fine with closing this.

Orion Poplawski <opoplawski>
Thu 30 Jun 2016 06:47:51 PM UTC, comment #7: 

More discussion from #octave:

There is something to be said about "want a way to install a package from an unpacked source directory" but this is not the way. Looking into Debian's "install-pkg" script from octave-pkg-dev there is extra logic required even with this patch applied.

So should we fix pkg install so that it installs from unpacked and patched sources, in which case there's more issues (not clear why which ones, we need details from downstream on this), or should we just remove this broken feature?

I'm leaning towards the later and instead of patching Octave to support this, it might be more efficient to add the required 'tar cfz' command to the distro package build tools.

Carnë Draug <carandraug>
Group Member
Thu 30 Jun 2016 05:55:56 PM UTC, comment #6: 

Discussing this on #octave, I just want to point out that both Debian and Fedora carry this patch in the octave builds for those distributions (and derivatives).

The purpose of the patch is to allow installing a package from an unpacked source directory. This makes "pkg install" behave like make, as in "wget thing.tar.gz; tar xf thing.tar.gz; cd thing; octave --eval 'pkg install .'".

I think it is acceptable if we decide that we don't want pkg to behave this way, and only be able to install packages from a packed tarball. But distribution packagers are still going to want a way to install a package from an unpacked source directory.

Mike Miller <mtmiller>
Group Member
Thu 30 Jun 2016 05:52:35 PM UTC, comment #5: 

It should only be accepting tarballs

pkg could also install from .zip files (e.g., the symbolic-...-bundle.zip files distributed by Colin).
Is it the intention to also drop that feature?

Philip Nienhuis <philipnienhuis>
Group Member
Thu 30 Jun 2016 03:01:19 PM UTC, comment #4: 

This looks like one of those undocumented features of pkg. I'm not sure what you mean with installing from directories. Did 'pkg install' used to support installing a package from an unpacked package?

Also, we rewrote pkg build from scratch recently. It should only be accepting tarballs.

Finally, what do you use this feature for?

Carnë Draug <carandraug>
Group Member
Thu 30 Jun 2016 01:01:01 PM UTC, comment #3: 

Orion, do you know if this is still an issue?  If so, could you please explain what you mean by "works with directories"?  What command do you issue, what should happen, and what does happen (without the patch)?

Lachlan Andrew <lachlan>
Sat 28 Dec 2013 11:35:53 PM UTC, comment #2: 

The pkg rewrite broke handling of directories.  The updated patch fixes.  One problem in the current code is that exist(path,'file') will return true if path is a file or a directory.

(file #30124)

Orion Poplawski <opoplawski>
Fri 12 Oct 2012 05:29:23 PM UTC, comment #1: 

Hi

thank you for your patch.

We are actually preparing a code sprint for pkg next month. Must make sure we take a look at this at that time. I actually never used the build option and seems we had develop something similar on OF, the releasePKG function. These two should probably be merged.

Carnë

Carnë Draug <carandraug>
Group Member
Fri 18 Mar 2011 04:11:03 PM UTC, original submission:  

With the changing of exist(,'file') behavior (presumably to match matlab) so exist('directory','file') returns 7, pkg build/install no longer works with directories. The attached patch fixes and supports the use of full directory paths.

Might also be good if the help for pkg build indicated that it works with directories.

Orion Poplawski <opoplawski>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #22955:  octave-3.4.0-pkgbuilddir.patch added by opoplawski (1KiB - text/x-patch - patch to fix pkg.m handling of directories.)

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by lachlan (Posted a comment)
  • -email is unavailable- added by carandraug (Posted a comment)
  • -email is unavailable- added by jordigh (Updated the item)
  • -email is unavailable- added by opoplawski (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 11 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-08-25 siko1056 StatusPostponed Wont Fix
        Open/ClosedOpen Closed
    2016-12-19 mtmiller Dependencies- bugs #49880 is dependent
    2016-11-16 mtmiller CategoryLibraries Octave Function
        Severity3 - Normal 2 - Minor
        StatusPatch Submitted Postponed
        Release3.4.0 dev
    2013-12-28 opoplawski Attached File- Added octave-3.8.0-pkgbuilddir.patch, #30124
    2012-02-11 rik5 Dependencies- bugs #32451 is dependent
    2011-04-02 jordigh StatusNone Patch Submitted
    2011-03-18 opoplawski Attached File- Added octave-3.4.0-pkgbuilddir.patch, #22955

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code