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

[dev.icinga.com #4955] [patch] Build issue, gnu-ism in find command line #1228

Closed
icinga-migration opened this issue Oct 25, 2013 · 3 comments
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Oct 25, 2013

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

Created by sthen on 2013-10-25 16:04:33 +00:00

Assignee: elippmann
Status: Resolved (closed on 2013-11-20 15:16:49 +00:00)
Target Version: 1.10.1
Last Update: 2013-11-20 15:18:01 +00:00 (in Redmine)

Icinga Version: 1.10.0
Icinga Web Version: 1.10.0
IDO Version: 1.10.0
OS Version: OpenBSD 5.4-current
DB Type: MySQL
DB Version: 5.1.72
Browser Version: chromium-29.0.1547.76

find -wholename used in lib/agavi/Makefile.in is nonstandard, please use -path instead as in the attached patch. Findutils documentation says "-wholename pattern See -path. This alternative is less portable than -path."

(assignee+target version as suggested by dnsmichi).

Attachments

Changesets

2013-11-20 15:13:30 +00:00 by sthen b4450b7

Build/find: Fix using -wholename which is less portable than -path

Signed-off-by: Eric Lippmann <eric.lippmann@netways.de>

fixes #4955

2013-12-09 15:47:34 +00:00 by sthen b75e220

Build/find: Fix using -wholename which is less portable than -path

Signed-off-by: Eric Lippmann <eric.lippmann@netways.de>

fixes #4955
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 20, 2013

Updated by elippmann on 2013-11-20 15:15:59 +00:00

  • Status changed from New to Assigned
  • Assigned to changed from mfrosch to elippmann
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 20, 2013

Updated by sthen on 2013-11-20 15:16:49 +00:00

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

Applied in changeset b4450b7.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Nov 20, 2013

Updated by elippmann on 2013-11-20 15:18:01 +00:00

Hi sthen,

Thanks for the patch.

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.