Bug 227802 - dns/maradns: Update 2.0.15
Summary: dns/maradns: Update 2.0.15
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: Mathieu Arnold
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-27 07:59 UTC by takefu
Modified: 2018-05-03 14:06 UTC (History)
0 users

See Also:


Attachments
maradns-2.0.15.patch (7.25 KB, patch)
2018-04-27 07:59 UTC, takefu
no flags Details | Diff
maradns-2.0.15.patch2 (6.11 KB, patch)
2018-05-01 07:19 UTC, takefu
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description takefu 2018-04-27 07:59:48 UTC
Created attachment 192837 [details]
maradns-2.0.15.patch

Update to 2.0.15

fix
  Shebangfix warning when CONV option is not used.
  MAKE_ENV deleted.
  Delete sourceforge from MASTER_SITES.
Comment 1 Mathieu Arnold freebsd_committer freebsd_triage 2018-04-27 09:44:40 UTC
Why rename bind2csv2 to bind2csv2.py ?

Also, please, do not rewrite all the Makefile for absolutely no reason, it makes it very hard to see what you are trying to do, and obfuscate everything.
Comment 2 Mathieu Arnold freebsd_committer freebsd_triage 2018-04-27 09:47:30 UTC
For example, this:



-	${INSTALL_MAN} ${WRKSRC}/doc/en/man/*.1 ${STAGEDIR}${MANPREFIX}/man/man1
-	${INSTALL_MAN} ${WRKSRC}/doc/en/man/*.8 ${STAGEDIR}${MANPREFIX}/man/man8
-	${INSTALL_MAN} ${WRKSRC}/doc/en/man/*.5 ${STAGEDIR}${MANPREFIX}/man/man5
+.for i in askmara.1 bind2csv2.1 getzone.1 fetchzone.1 mqhash.1\
+	maradns.8 zoneserver.8 duende.8\
+	mararc.5 csv1.5 csv2.5 csv2_txt.5 Deadwood.1
+	${INSTALL_MAN} ${WRKSRC}/doc/en/man/${i}\
+		${STAGEDIR}${MANPREFIX}/man/${i:C|.*\.|man|}/${i:C|\.*||}
+.endfor

Is absolutely ludicrous, the first one was very clear and easy to read, your new version is completely obfuscated.
Comment 3 takefu 2018-05-01 06:28:35 UTC
(In reply to Mathieu Arnold from comment #1)

bind2csv2.py is my mistake.
Makefile broke the line to accommodate 80 characters per line, but if you do not like it, would you like to change it without changing it?
Comment 4 takefu 2018-05-01 06:58:40 UTC
(In reply to Mathieu Arnold from comment #2)

Obfuscation. It can not be helped thought so.
Just as I reviewed it now, this change is already unnecessary.
So I will patch it again.

There seemed to be reasons for rewriting to such a patch, but it seems that it got stale.
Comment 5 takefu 2018-05-01 07:19:15 UTC
Created attachment 192954 [details]
maradns-2.0.15.patch2
Comment 6 commit-hook freebsd_committer freebsd_triage 2018-05-03 14:06:16 UTC
A commit references this bug:

Author: mat
Date: Thu May  3 14:05:55 UTC 2018
New revision: 468929
URL: https://svnweb.freebsd.org/changeset/ports/468929

Log:
  Update to 2.0.15.

  PR:		227802
  Submitted by:	new maintainer
  Sponsored by:	Absolight

Changes:
  head/dns/maradns/Makefile
  head/dns/maradns/distinfo
  head/dns/maradns/files/patch-configure
  head/dns/maradns/files/patch-tools__Makefile