spacer make - Bugs: bug #30653, intermediate files incorrectly...

Group
  • Main
  • Homepage
  • Download
  • Docs
  • Support
  • Mailing lists
  • Source code
  • Bugs
  • Patches
  • News
 
 

bug #30653: intermediate files incorrectly pruned in parallel builds

Submitted by:  Jeremy Devenport <jeremyd>
Submitted on:  Thu Aug 5 01:24:56 2010  
 
Severity: 3 - NormalItem Group: Bug
Status: FixedPrivacy: Public
Assigned to: Paul D. Smith <psmith>Open/Closed: Closed
Component Version: 3.82Operating System: Any
Fixed Release: 4.0Triage Status: None

Add a New Comment (spacer Rich Markup):
   

You are not logged in

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

 

Mon Jan 6 20:10:33 2014, comment #5:

I'm personally not invested either way since this particular bug fix is short enough to not be subject to copyright anyway (it falls under fair use exceptions), so you could just create your own patch file with that change in it (the one additional line to remake.c) without asking permission at all and that's perfectly fine.

However just to note, a patch is clearly a derived work of the GNU make codebase and as such (if it was subject to copyright requirements, and barring other agreements such as copyright assignment etc.) it would be jointly copyright between the FSF and Jeremy D. and the only way the license could be modified would be with the unanimous agreement of both parties.

The applicable license, unless/until a change was agreed, would be the one under which the code was licensed when Jeremy D. created his patch. Assuming he was working with GNU make 3.82, the license is GPLv3.

It's not possible to backport changes from newer versions of GNU make, with a newer license, to older versions of GNU make, with an older license, then release the whole thing under the older license. That's not how it works...

Paul D. Smith <psmith>
spacer spacer
Mon Jan 6 19:48:43 2014, comment #4:

The version committed appears to be my patch (with a cleanup to the code comment and the test case condensed to use fewer lines). I'm fine with making my patch available as GPL2+.

Jeremy Devenport <jeremyd>
Sat Jan 4 19:05:54 2014, comment #3:

Jeremy and Kamil,

Can you please comment as to the licensing requirements of your patches? Are they available as GPL2+ or GPL3? (ie, will you allow me to apply them to make 3.8.1 as GPL2+).

If you specified in the patch, I apologize as I do not want to look at them prior to knowing that they are not GPL3+.

Which version was actually committed?

Jeremy Huddleston Sequoia <jeremyhu>
Mon Sep 10 02:36:58 2012, comment #2:

Fix applied. Thank you both for the useful test cases and debugging work.

Paul D. Smith <psmith>
spacer spacer
Thu Aug 5 08:47:09 2010, comment #1:

I have also encountered this bug and debugged it a little. I posted my comments to the bug-make list:

lists.gnu.org/archive/html/bug-make/2010-07/msg00155.html

Kamil Mierzejewski <kmierzejewski>
Thu Aug 5 01:24:56 2010, original submission:

make 3.82 sometimes fails to make intermediate (or secondary) files during parallel builds. I see that there are a couple bug reports with similar titles but this particular bug seems to have been introduced in 3.82 (and is still present in CVS).

I'm attaching a patch that adds a test that demonstrates the bug along with a fix that seems to work. I'm not very familiar with the code so I'm not sure that my fix doesn't break something else.

Jeremy Devenport <jeremyd>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #21146:  parallel_intermediates.patch added by kmierzejewski (674B - application/octet-stream - I also encountered this bug. I came up with an alternative solution. I'm not sure if it breaks anything as well.)
file #21144:  make-intermediate-parallel-bug.patch added by jeremyd (2kB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jeremyhu (Posted a comment)
  • -unavailable- added by psmith (Posted a comment)
  • -unavailable- added by kmierzejewski (Updated the item)
  • -unavailable- added by jeremyd (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 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Sep 10 02:36:58 2012psmithStatusNonespacer " />Fixed
      Assigned toNonespacer " />psmith
      Open/ClosedOpenspacer " />Closed
      Component Version4.0spacer " />3.82
      Fixed ReleaseNonespacer " />4.0
    Thu Aug 5 08:45:04 2010kmierzejewskiAttached File-spacer " />Added parallel_intermediates.patch, #21146
    Thu Aug 5 01:24:56 2010jeremydAttached File-spacer " />Added make-intermediate-parallel-bug.patch, #21144

    spacer


    Source
    Code
    Copyright © 2015  Free Software Foundation, Inc.
    Verbatim copying and distribution of this entire article is permitted in any medium, provided this notice is preserved.
    The Levitating, Meditating, Flute-playing Gnu logo is a GNU GPL'ed image provided by the Nevrax Design Team.

    Powered by Savane 3.1-cleanup
    gipoco.com is neither affiliated with the authors of this page nor responsible for its contents. This is a safe-cache copy of the original web site.