bugmake - Bugs: bug #50823, MAKEFILE_LIST contains wrong file...

 
 

bug #50823: MAKEFILE_LIST contains wrong file name if file name contains dollar character

Submitter:  None
Submitted:  Tue 18 Apr 2017 08:16:02 AM UTC
   
 
Severity:  3 - Normal Item Group:  Bug
Status:  Fixed Privacy:  Public
Assigned to:  psmith Open/Closed:  Closed
Component Version:  4.2.1 Operating System:  POSIX-Based
Fixed Release:  4.3 Triage Status:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 04 Jun 2017 10:55:37 PM UTC, comment #2: 

I decided to use a different fix, that didn't have any backward-compat issues.

A fix has been pushed and will be in the next release.  Thank you for filing a report!

Paul D. Smith <psmith>
Group administrator
Tue 25 Apr 2017 02:56:13 PM UTC, comment #1: 

Patch to fix #50823, see attached file.

(file #40504)

Dominik Muth <dominik_muth>
Tue 18 Apr 2017 08:16:02 AM UTC, original submission:  

To reproduce:

$ echo -e 'all:\n\techo $(value MAKEFILE_LIST)' > /tmp/foo\$bar.mk
$ ./make -f '/tmp/foo$bar.mk'
echo  /tmp/fooar.mk
/tmp/fooar.mk

I think this is inconsistent and contradicts the documentation, which states "MAKEFILE_LIST Contains the name of each makefile that is parsed by make, in the order in which it was parsed."

See also http://stackoverflow.com/q/43466130/2965738

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #40504:  0001-Make-MAKEFILE_LIST-a-recursively-expanded-variable.patch added by dominik_muth (3KiB - application/octet-stream - Patch to fix #50823)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by psmith (Posted a comment)
  • -email is unavailable- added by dominik_muth (Updated 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 logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-06-04 psmith StatusNone Fixed
        Assigned toNone psmith
        Open/ClosedOpen Closed
        Fixed ReleaseNone 4.3
    2017-04-25 dominik_muth Attached File- Added 0001-Make-MAKEFILE_LIST-a-recursively-expanded-variable.patch, #40504

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code