bugmake - Bugs: bug #24509, doc for wildcard expansion in...

 
 

bug #24509: doc for wildcard expansion in commands could be clearer

Submitter:  Martin Dorey <mdorey>
Submitted:  Thu 09 Oct 2008 06:20:47 PM UTC
   
 
Severity:  3 - Normal Item Group:  Documentation
Status:  Fixed Privacy:  Public
Assigned to:  psmith Open/Closed:  Closed
Component Version:  4.0 Operating System:  None
Fixed Release:  3.82 Triage Status:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 02 Aug 2009 04:19:21 PM UTC, comment #1: 

Thanks Martin.

Paul D. Smith <psmith>
Group administrator
Thu 09 Oct 2008 06:20:47 PM UTC, original submission:  

Harald Bergmann notes that in one place we say "the shell is responsible for wildcard expansion" where later we say "[if you use .o in a] command, wildcard expansion *will take place at that time" (my emphasis on will).  Well, perhaps it will, perhaps it will not - that depends on the shell, as noted earlier.  Further, any expansion doesn't take place at the time make sees the command, which the above wording could reasonably be interpreted to suggest, but only when make runs the shell and the shell runs the command.

On the mailing list, I suggested [

I'm not sure I've understood.  Perhaps rewording the second stanza like
this would address your concern?

"However, if you use the value $(objects) in a target or prerequisite, wildcard expansion will take place there.  If you use the value $(objects) in a command, the shell may perform wildcard expansion when the command runs."

] and the OP replied (privately) [

Hi Martin,

I am sure you did!
Your proposal is OK!

Best regards,
Harald

].

That proposal is not quite the same as the patch, from CVS, that I'm attaching here, because the wording has seemingly moved on (using the word "recipe" instead of "command") and because there's quoting in the original that didn't survive the email, leading me to suggest my own $() "quoting", above, which I abandon here in favor of the existing style.

I reproduce it as evidence that my first paragraph here is an accurate summary of the OP's gripe.

Martin Dorey <mdorey>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #16640:  make.texi.patch added by mdorey (855B - application/octet-stream)

 

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 mdorey (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 logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-07-28 psmith Fixed Release4.0 3.82
    2009-08-02 psmith StatusNone Fixed
        Assigned toNone psmith
        Open/ClosedOpen Closed
        Fixed ReleaseNone 4.0
    2008-10-09 mdorey Attached File- Added make.texi.patch, #16640

    Back to the top

    Powered by Savane 3.14-8eb0.
    Corresponding source code