patchFreeciv - Patches: patch #3468, Metaticket: Clang clean build

 
 
Show feedback again

patch #3468: Metaticket: Clang clean build

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Tue 31 Jul 2012 09:59:27 PM UTC  
 
Category: bootstrapPriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: 2.6.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Thu 16 Jan 2014 07:57:52 PM UTC, comment #6:

> We can aim for Clang-clean with all options like --enable
> sys-lua enabled to remove as much upstream trouble as possible?


The main upstream problem I were thinking about is that MagickWand-config gives bad CPPFLAG for us.

Marko Lindqvist <cazfi>
Project Administrator
Sat 11 Jan 2014 10:49:17 AM UTC, comment #5:

> Some things seem like derived from upstreams in a way that we
> really depend on upstream to fix them.

We can aim for Clang-clean with all options like --enable-sys-lua enabled to remove as much upstream trouble as possible?

Jacob Nevins <jtn>
Project Administrator
Sat 11 Jan 2014 07:57:15 AM UTC, comment #4:

patch #4421 is now about silencing the evil -Wtautological-compare, and after that the remaining errors seem rather relevant. Not that I'm sure we can reach clang-clean build in freeciv itself ( -> we cannot decide the schedule). Some things seem like derived from upstreams in a way that we really depend on upstream to fix them.

Marko Lindqvist <cazfi>
Project Administrator
Thu 21 Feb 2013 02:51:55 PM UTC, comment #3:

Looking at the clang warnings lists I'm inclined to say that clang clean build should not be goal for us, or at least many warning types should be silenced with -Wno- -options instead of reworking the code to avoid the warning. For instance the "tautological compare" warnings because macro expansion often leads to value being compared to itself seem more like something that compiler should use for optimization, not to nag developers about. To work around them would make code much more complicated and likely introduce a lot of real bugs while fixing none.

Marko Lindqvist <cazfi>
Project Administrator
Wed 30 Jan 2013 10:39:53 AM UTC, comment #2:

> I don't think anybody wants to fight warnings from lua-5.1 and
> friends when we hopefully get lua-5.2 to use before 2.5 (another
> release goal?)


As lua gets build first, it's rather hard to work on this before lua is updated. There seems to be no progress on that front. I think it's unrealistic to expect us to take project of clang warnings fixing between lua update and 2.5.0 release.
So, let's update lua for 2.5 and target clang warning fixing to 2.6.

Marko Lindqvist <cazfi>
Project Administrator
Tue 31 Jul 2012 10:13:58 PM UTC, comment #1:

I don't think anybody wants to fight warnings from lua-5.1 and friends when we hopefully get lua-5.2 to use before 2.5 (another release goal?)

Marko Lindqvist <cazfi>
Project Administrator
Tue 31 Jul 2012 09:59:27 PM UTC, original submission:

Clang warnings came too late to be considered for systematic fixing in S2_3. However, getting all of them resolved should be release goal at some point. At least for 2.5, 2.4 would be nice, but I don't want to add new goals this late if they end postponing the release - I think that we should commit any fixes to S2_4 too (those that turn out to be real bugs naturally even to S2_3) but accept the fact if they are not all fixed by the time 2.4.0 is otherwise ready.

Marko Lindqvist <cazfi>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Digest:
   bug dependencies, patch dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jtn (Posted a comment)
  • -unavailable- added by cazfi (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 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 16 Jan 2014 08:05:37 PM UTCcazfiDependencies-=>Depends on bugs #21493
    Thu 16 Jan 2014 07:58:13 PM UTCcazfiDependencies-=>Depends on bugs #21492
    Sat 11 Jan 2014 07:46:24 AM UTCcazfiDependencies-=>Depends on patch #4421
    Tue 31 Dec 2013 12:25:04 PM UTCjtnDependencies-=>Depends on patch #4373
    Tue 31 Dec 2013 12:24:37 PM UTCjtnDependencies-=>Depends on bugs #20776
    Wed 30 Jan 2013 10:39:53 AM UTCcazfiPlanned Release2.5.0=>2.6.0
    Tue 31 Jul 2012 10:20:14 PM UTCjtnDependencies-=>Depends on bugs #18872
    Tue 31 Jul 2012 10:19:59 PM UTCjtnDependencies-=>Depends on bugs #18979
    Tue 31 Jul 2012 10:19:30 PM UTCjtnDependencies-=>Depends on bugs #19307
    Tue 31 Jul 2012 10:13:58 PM UTCcazfiPlanned Release=>2.5.0
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup