Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various build fixups (missing includes, name clashes, ...) #1774

Closed
mc-butler opened this issue Oct 31, 2009 · 12 comments
Closed

Various build fixups (missing includes, name clashes, ...) #1774

mc-butler opened this issue Oct 31, 2009 · 12 comments
Labels
area: core Issues not related to a specific subsystem prio: blocker This problem will block progress ver: 4.7.0-pre4 Reproducible in version 4.7.0-pre4
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/1774
Reporter metux (@metux)
Keywords build, -Werror, includes, name-conflicts

There are several unclean things which break build in -Werror mode, eg: char* vs const char* mixups, missing includes, conflicting variable names, etc.

branch:1774_build_cleanups
[4a1e9c124c6028307be260bb8d2e0c0e516a2c1a]

@mc-butler
Copy link
Author

Changed by metux (@metux) on Oct 31, 2009 at 19:35 UTC (comment 1)

  • Severity changed from no branch to on review
  • Owner set to metux
  • Status changed from new to accepted

@mc-butler
Copy link
Author

Changed by metux (@metux) on Oct 31, 2009 at 19:43 UTC (comment 2)

  • Priority changed from major to blocker

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 2, 2009 at 9:43 UTC (comment 3)

  • Votes set to slavazanko

Branch for this ticket: 1774_build_cleanups

Looks good.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 2, 2009 at 9:43 UTC (comment 4)

  • Milestone changed from 4.7 to 4.7.0

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 2, 2009 at 10:03 UTC (comment 5)

  • Votes slavazanko deleted

My latest correct [f91d4aa1ac6e07ecb10b47abd7e642619b749036]

Changed include 'glib.h' to 'global.h' besause global.h file have some things for backward compability with glib (eg. including 'glibcompat.h')

Review.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 2, 2009 at 10:18 UTC (comment 6)

Also, branch rebased for more WorkingGuideLines-complicant

Added Ticket number in first commit of branch.

Initial [083d3546a6979c81e2a2a728b4b832fa6e2ed144]

@mc-butler
Copy link
Author

Changed by metux (@metux) on Nov 3, 2009 at 23:37 UTC (comment 7)

  • Description edited

Rebased to current master

@mc-butler
Copy link
Author

Changed by iNode (@iNode) on Nov 4, 2009 at 14:32 UTC (comment 8)

  • Votes set to iNode

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 4, 2009 at 15:35 UTC (comment 9)

  • Severity changed from on review to approved
  • Votes changed from iNode to iNode slavazanko

@mc-butler
Copy link
Author

Changed by metux (@metux) on Nov 9, 2009 at 12:48 UTC (comment 10)

  • Description edited
  • Severity changed from approved to merged

rebased and merged into master

@mc-butler
Copy link
Author

Changed by metux (@metux) on Nov 11, 2009 at 11:05 UTC (comment 11)

  • Status changed from accepted to testing
  • Resolution set to fixed

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Nov 11, 2009 at 13:42 UTC (comment 12)

  • Status changed from testing to closed
  • Votes changed from iNode slavazanko to commited-master
git log --pretty=oneline d740efc...de4e40d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: blocker This problem will block progress ver: 4.7.0-pre4 Reproducible in version 4.7.0-pre4
Development

No branches or pull requests

1 participant