Bug 241444 - graphics/openorienteering-mapper: Update to 0.9.0
Summary: graphics/openorienteering-mapper: Update to 0.9.0
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kai Knoblich
URL: https://www.openorienteering.org/news...
Keywords: buildisok
Depends on:
Blocks:
 
Reported: 2019-10-23 20:26 UTC by Nikolay Korotkiy
Modified: 2019-11-01 20:20 UTC (History)
2 users (show)

See Also:
sikmir: maintainer-feedback+


Attachments
Update to 0.9.0 (14.40 KB, patch)
2019-10-23 20:26 UTC, Nikolay Korotkiy
sikmir: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolay Korotkiy 2019-10-23 20:26:59 UTC
Created attachment 208541 [details]
Update to 0.9.0
Comment 1 Automation User 2019-10-23 21:26:18 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/91065379
Comment 2 Kai Knoblich freebsd_committer 2019-11-01 14:27:51 UTC
Hi Nikolay,

thank you for the update. It's soon ready to land into the ports tree.

I've only one question regarding the both patches that will be deleted:

- files/patch-translations_CMakeLists.txt
- files/patch-translations_translate__text__files.sh

It seems that sed(1) does its job fine (hence the removal of the patches I assume). I guess we can then remove also the following line from the Makefile:

> NLS_BUILD_DEPENDS=      gsed:textproc/gsed
Comment 3 Nikolay Korotkiy 2019-11-01 16:39:41 UTC
Yes, gsed should be removed.
Comment 4 commit-hook freebsd_committer 2019-11-01 20:16:38 UTC
A commit references this bug:

Author: kai
Date: Fri Nov  1 20:16:32 UTC 2019
New revision: 516264
URL: https://svnweb.freebsd.org/changeset/ports/516264

Log:
  graphics/openorienteering-mapper: Update to 0.9.0

  * Remove textproc/gsed from BUILD_DEPENDS for the NLS option and drop the
    related patches for it as sed(1) works fine.

  * It's also no longer required to use the deprecated "proj_api.h" from
    graphics/proj thus remove the workaround with CXXFLAGS that was introduced
    with r505194.

  * Rebase the patch for packaging/custom_install.cmake.in to avoid warnings
    while patching because the file has now less lines since the last release.

  Changelog:

  https://github.com/OpenOrienteering/mapper/releases/tag/v0.9.0

  PR:		241444
  Submitted by:	Nikolay Korotkiy <sikmir@gmail.com> (maintainer)

Changes:
  head/graphics/openorienteering-mapper/Makefile
  head/graphics/openorienteering-mapper/distinfo
  head/graphics/openorienteering-mapper/files/patch-packaging_custom__install.cmake.in
  head/graphics/openorienteering-mapper/files/patch-src_sensors_CMakeLists.txt
  head/graphics/openorienteering-mapper/files/patch-translations_CMakeLists.txt
  head/graphics/openorienteering-mapper/files/patch-translations_translate__text__files.sh
  head/graphics/openorienteering-mapper/pkg-plist
Comment 5 Kai Knoblich freebsd_committer 2019-11-01 20:20:11 UTC
(In reply to Nikolay Korotkiy from comment #3)

Thank you for your info. I removed textproc/gsed from the BUILD_DEPENDS for NLS and your patch has landed into the Ports tree!