Bug 236299

Created attachment 202617 [details]
Shell archive

Poudriere revealed that:
• gettext-tools is needed at built time
• more dependencies are needed to generate the manual page

I am on this...

Resurrect converters/pdf2djvu:
- Fetch the distribution file from GitHub
- Add USES=autoreconf:build and a bootstrap file; define the target pre-configure
- Make the manual page within the post-build target
- Add USE_CXXSTD=c++11

- Update to 0.9.12
- Add options COLOR_QUANTIZATION, DOCS, NLS, MULTITHREADING, and XMP
- Add missing dependencies

While here:
- Add LICENSE_FILE
- Patch tools/install-manpages for proper echoing (no functional change)
- Remove the port name from COMMENT to follow the guidelines of the handbook
- Strip the executable within the post-install target

Reference: https://www.freshports.org/converters/pdf2djvu

Created attachment 202622 [details]
Shell archive

Created attachment 202623 [details]
Poudriere log

Resurrect converters/pdf2djvu:
- Fetch the distribution file from GitHub
- Add USES=autoreconf:build and a bootstrap file; define the target pre-configure
- Make the manual page within the post-build target
- Add USE_CXXSTD=c++11

- Update to 0.9.12
- Add missing dependencies
- Add CPPFLAGS and LDFLAGS for proper OpenMP activation
- Add options COLOR_QUANTIZATION, DOCS, NLS, MULTITHREADING, and XMP

While here:
- Add LICENSE_FILE
- Patch tools/install-manpages for proper echoing (no functional change) *
- Remove the port name from COMMENT to follow the guidelines of the handbook
- Strip the executable within the post-install target

Reference:
https://www.freshports.org/converters/pdf2djvu
* https://github.com/jwilk/pdf2djvu/commit/5f7e01324b6b982856505ebca764c8553cf984e1

Created attachment 202626 [details]
Shell archive

Created attachment 202627 [details]
Poudriere log

Comment 9 Koichiro Iwao 2019-03-09 03:29:56 UTC

Could you submit a patch versus the snapshot at the point just before the port deleted?

(In reply to Koichiro Iwao from comment #9)

I will certainly try, at least.

Comment 11 Koichiro Iwao 2019-03-11 00:45:04 UTC

diff versus r471891 needed.

(In reply to Koichiro Iwao from comment #11)

I am on my way.

Created attachment 202788 [details]
Patch file generated with svn diff

Done.

Comment 14 Koichiro Iwao 2019-03-11 01:08:06 UTC

Thanks, I'm testing.

Comment 15 commit-hook 2019-03-11 02:32:39 UTC

A commit references this bug:

Author: meta
Date: Mon Mar 11 02:32:06 UTC 2019
New revision: 495325
URL: https://svnweb.freebsd.org/changeset/ports/495325

Log:
  Readd and rework convertes/pdf2djvu

  This port was deleted due to no public distfiles. Now upstream distributes
  tarballs on GitHub [1].

  - Update to 0.9.12
  - Add USES=autoreconf:build and a bootstrap file; define the target pre-configure
  - Make the manual within the post-build target
  - Add USE_CXXSTD=c++11
  - Add options COLOR_QUANTIZATION, NLS, MULTITHREADING, XMP
  - Add LICENSE_FILE
  - Patch tools/install-manpages for proper echoing (no functional change)
  - Remove the port name from COMMENT to follow the guidelines of the handbook
  - Strip the executable within the post-install target
  - Submitter is the new MAINTAINER

  [1] https://github.com/jwilk/pdf2djvu

  PR:		236299
  Submitted by:	Samy Mahmoudi <samy.mahmoudi@gmail.com>

Changes:
  head/MOVED
  head/converters/Makefile
  head/converters/pdf2djvu/
  head/converters/pdf2djvu/Makefile
  head/converters/pdf2djvu/distinfo
  head/converters/pdf2djvu/files/
  head/converters/pdf2djvu/files/bootstrap
  head/converters/pdf2djvu/files/patch-tools_install-manpages

Comment 16 Koichiro Iwao 2019-03-11 02:34:22 UTC

Committed with minor changes, thanks!

(In reply to Koichiro Iwao from comment #16)

Thank you!