Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2620] Compatibility problem in module/idoutils/db.c #979

Closed
icinga-migration opened this issue May 17, 2012 · 4 comments
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented May 17, 2012

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2620

Created by crfriend on 2012-05-17 21:49:38 +00:00

Assignee: Tommi
Status: Resolved (closed on 2012-06-12 16:13:04 +00:00)
Target Version: 1.7.1
Last Update: 2014-12-08 14:46:39 +00:00 (in Redmine)


In building the release of Icinga 1.7.0 on gcc 2.95.3 I encountered a fatal error in db.c at line 1322. The offending statement was a declaration intermixed with statements.

The attached patch corrects portability issues, not logic errors. Some compilers understand this idiom, but some don't.

Metadiscussion: Should there be a "category" for "Portability" or just leave it as "Other"?

Attachments

Changesets

2012-05-30 19:17:45 +00:00 by Tommi 09ab12ed595c37e7897543f558cd8bf1aa26fe74

idoutils: fix small compiler issues #2620
refs #2620

2012-06-12 15:41:45 +00:00 by Tommi 1cda0b3

idoutils: fix small compiler issues #2620
refs #2620

Conflicts:

	module/idoutils/src/db.c

history below was broken on the cherry-pick, so resolved it for a bugfix
release.

2012-10-20 11:50:16 +00:00 by Tommi 8403979b2a496834652381d6b3ed0c5ebe15b190

idoutils: fix small compiler issues #2620
refs #2620
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented May 18, 2012

Updated by mfriedrich on 2012-05-18 09:52:57 +00:00

  • Project changed from Core, Classic UI, IDOUtils to 18
  • Category deleted Other
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented May 30, 2012

Updated by Tommi on 2012-05-30 19:26:46 +00:00

  • Category set to 24
  • Status changed from New to Feedback
  • Assigned to set to Tommi
  • Done % changed from 0 to 100

reorder of definition/assignments done in changeset 09ab12ed595c37e7897543f558cd8bf1aa26fe74

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 12, 2012

Updated by mfriedrich on 2012-06-12 16:13:04 +00:00

  • Status changed from Feedback to Resolved
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Dec 8, 2014

Updated by mfriedrich on 2014-12-08 14:46:39 +00:00

  • Project changed from 18 to Core, Classic UI, IDOUtils
  • Category changed from 24 to IDOUtils
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.