bugmake - Bugs: bug #31360, cannot handle path expansion...

 
 

bug #31360: cannot handle path expansion correctly

Submitted by:  wilson tien <wilsontien>
Submitted on:  Mon 18 Oct 2010 09:11:47 AM UTC  
 
Severity: 3 - NormalItem Group: Bug
Status: Not A BugPrivacy: Public
Assigned to: NoneOpen/Closed: Closed
Component Version: 3.81Operating System: POSIX-Based
Fixed Release: NoneTriage 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 Mar 2012 05:15:51 PM UTC, comment #1:

This is not a Linux issue or a make issue. The POSIX standard, and every UNIX system I'm aware of, requires that the root directory (like all other directories) have a ".." entry and that "/../." is identical to "/.". On every UNIX-based system that has ever existed if you "cd /" then "cd .." this does not fail, you're still in the root directory.

Similarly, running "cat /../../../foo" will cat out the file "/foo", not fail.

Unless I'm misunderstanding your report, make is simply behaving like every other UNIX/POSIX utility and this is the correct behavior.

Paul D. Smith <psmith>
Project Administrator
Mon 18 Oct 2010 09:11:47 AM UTC, original submission:

in Xen 4.0.1 build, there is a line in the Makefile
-include ../../../../.config
which follows BSD make convention, should not generate error message even if the included file does not exist. However, it continues to bug the build process. The reason is simple. include file will be concat with include_directories[], and try to see whether we can find it somewhere in the loop. However, /usr/include/../../../../.config will resolve to /.config even the original intent is not that since 'cd ..' at '/' will still be '/'. If there is a '/.config' at '/', then -include will simply fail.
'../' in Makefile should really be just 'parent dir' and if there is no parent dir, it should simply FAIL. IMHO.

wilson tien <wilsontien>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #21703:  read.c.patch added by wilsontien (2KiB - text/x-patch)
file #21704:  testopen.out added by wilsontien (275B - application/octet-stream)
file #21705:  testopen.c added by wilsontien (768B - text/x-csrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

     

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

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 04 Mar 2012 05:15:51 PM UTCpsmithStatusNone=>Not A Bug
      Open/ClosedOpen=>Closed
    Mon 18 Oct 2010 09:11:47 AM UTCwilsontienAttached File-=>Added read.c.patch, #21703
      Attached File-=>Added testopen.out, #21704
      Attached File-=>Added testopen.c, #21705

    Back to the top


    Powered by Savane 3.1-cleanup1