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

 
 

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

Submitted by:  None
Submitted on:  Tue 18 Apr 2017 08:16:02 AM UTC  
 
Severity:  3 - Normal Item Group:  Bug
Status:  Fixed Privacy:  Public
Assigned to:  Paul D. Smith <psmith> Open/Closed:  Closed
Component Version:  4.2.1 Operating System:  POSIX-Based
Fixed Release:  4.3 Triage Status:  None

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

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>
Project AdministratorIn charge of this item.
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)
  •  

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

    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.7