bugGNU Octave - Bugs: bug #46504, unpack fails on urls

 
 

bug #46504: unpack fails on urls

Submitter:  Muhali <muhali>
Submitted:  Mon 23 Nov 2015 07:49:50 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed 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
   

Tue 24 Nov 2015 10:56:54 PM UTC, comment #5: 

Small typo fixed here (http://hg.savannah.gnu.org/hgweb/octave/rev/bb219f1d92db) and unpack now passes all tests.

Rik <rik5>
Group administrator
Tue 24 Nov 2015 09:36:58 PM UTC, comment #4: 

"make check" now generates 3 FAILs for unpack, related to (AFAICS) expecting an error when in fact unpack.m just emits an (incomplete) warning

Only after next weekend I can look into this in more detail

Philip Nienhuis <philipnienhuis>
Group Member
Tue 24 Nov 2015 07:50:48 PM UTC, comment #3: 

I restored the ability to have URLs as inputs to unpack.  See this cset (http://hg.savannah.gnu.org/hgweb/octave/rev/69aeef6736df).

Rik <rik5>
Group administrator
Tue 24 Nov 2015 07:05:21 PM UTC, comment #2: 

unpack works if you remove the call to glob (l. 89) that was introduced in the mentioned changeset. urls are not understood by glob.

Muhali <muhali>
Tue 24 Nov 2015 06:44:17 PM UTC, comment #1: 

(just a hunch, I could be completely wrong)
Did you notice this only recently, and did it work with earlier dev versions?
If yes, chances are that it is connected to bug #46216

I see little else in the changeset you refer to that could give rise to this new bug.

Philip Nienhuis <philipnienhuis>
Group Member
Mon 23 Nov 2015 07:49:50 PM UTC, original submission:  

unpack does not work with urls anymore:

error: unpack: file "http://mirrors.ctan.org/support/tex2rtf/rtfutils/rtftohtml/README.gz" not found
error: called from
    unpack at line 91 column 7

The error is introduced in this changeset.

Muhali <muhali>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by muhali (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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-11-24 rik5 StatusNeed Info Fixed
        Open/ClosedOpen Closed
    2015-11-24 philipnienhuis StatusNone Need Info

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code