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

[dev.icinga.com #3575] make dist-* does not include lib/config/config_parser.h #431

Closed
icinga-migration opened this issue Jan 26, 2013 · 19 comments
Labels
bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

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

Created by Meier on 2013-01-26 12:24:04 +00:00

Assignee: gbeutner
Status: Resolved (closed on 2013-09-30 07:35:19 +00:00)
Target Version: 0.0.3
Last Update: 2013-09-30 07:35:19 +00:00 (in Redmine)

Icinga Version: 0.0.2

When I run make dist-gzip the resulting tar.gz does not include lib/config/config_parser.h. Automake seems to automatically exclude files from BUILT_SOURCES for DIST_COMMON even if it is added to libconfig_la_SOURCES.

Platform: Fedora 18 x86_64, automake-1.12.2-5.fc18.noarch

Workaround: manually add config_parser.h to DIST_COMMON in lib/config/Makefile(.in) after running autoconf

Changesets

2013-09-26 16:58:02 +00:00 by (unknown) f1bc61c

Add config_parser.h to EXTRA_DIST.

fixes #3575

2013-09-27 08:27:50 +00:00 by (unknown) 41e6d50

Revert "Add config_parser.h to EXTRA_DIST."

This reverts commit f1bc61c1885531690f4c237446f19aaced8cc62b.

refs #3575

2013-09-30 07:15:22 +00:00 by (unknown) aa99061

Fix: make dist-* does not include lib/config/config_parser.h

Fixes #3575

Relations:

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-01-28 09:51:02 +00:00

  • Target Version set to Milestone 2

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-01-28 10:45:03 +00:00

Hm, looks like Fedora 18's flex uses "config_parser.hh" (rather than "config_parser.h") as the header file name.

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-01-29 08:40:38 +00:00

Correction: This is actually related to this automake change: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=7648

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-02-03 22:25:49 +00:00

  • Target Version changed from Milestone 2 to 107

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-02-08 22:49:42 +00:00

  • Priority changed from Normal to Low

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-03-20 11:16:38 +00:00

there's a new revision available for fc18 http://koji.fedoraproject.org/koji/buildinfo?buildID=397188
the newer 1.13.1 release is available in fc19 only. i think we should reschedule the issue towards m4 (or final release tests). and if fc18 stays broken, we'll mark it somewhere in the README.

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-03-20 11:17:12 +00:00

Well, JFDI.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-03-20 11:25:43 +00:00

  • Target Version changed from 107 to 88

@icinga-migration
Copy link
Author

Updated by gbeutner on 2013-05-14 07:50:22 +00:00

  • Target Version deleted 88

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-07-30 20:21:19 +00:00

  • Target Version set to 130

happens with jessie currently as well.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-23 09:40:31 +00:00

  • Target Version changed from 130 to 0.0.3

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-26 16:58:09 +00:00

  • Status changed from New to Assigned
  • Assigned to changed from gbeutner to mfriedrich
  • Icinga Version set to 0.0.2

@icinga-migration
Copy link
Author

Updated by Anonymous on 2013-09-26 17:06:14 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset f1bc61c.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-27 08:16:32 +00:00

  • Category set to Installation
  • Status changed from Resolved to Assigned
  • Assigned to changed from mfriedrich to gbeutner
  • Priority changed from Low to Normal
  • Done % changed from 100 to 0

@icinga-migration
Copy link
Author

Updated by Anonymous on 2013-09-27 13:57:15 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset icinga2-ido:f1bc61c1885531690f4c237446f19aaced8cc62b.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-27 14:13:42 +00:00

  • Status changed from Resolved to Assigned
  • Done % changed from 100 to 0

@icinga-migration
Copy link
Author

Updated by Anonymous on 2013-09-28 15:00:14 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset icinga2-plugins:f1bc61c1885531690f4c237446f19aaced8cc62b.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-29 12:01:25 +00:00

  • Status changed from Resolved to Assigned

sorry, my bad. redmine inherits the repository from the parent project, but it does not display it via menu - duplicating the repository in all sub projects manually makes sense - until they update issues from git commits as separate repository each.

@icinga-migration
Copy link
Author

Updated by Anonymous on 2013-09-30 07:35:19 +00:00

  • Status changed from Assigned to Resolved

Applied in changeset aa99061.

@icinga-migration icinga-migration added bug Something isn't working Installation labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 0.0.3 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant