Bug 211131 - math/units: Update to 2.13
Summary: math/units: Update to 2.13
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Dmitry Marakasov
URL:
Keywords: easy, needs-qa, patch
Depends on:
Blocks:
 
Reported: 2016-07-14 23:47 UTC by jharris
Modified: 2016-07-26 15:57 UTC (History)
1 user (show)

See Also:


Attachments
update math/units to 2.13 (760 bytes, patch)
2016-07-14 23:47 UTC, jharris
jharris: maintainer-approval+
Details | Diff
patch without 0x0d (Carriage Returns) - thanks, Apple OS X! (712 bytes, patch)
2016-07-21 18:16 UTC, jharris
jharris: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jharris 2016-07-14 23:47:28 UTC
Created attachment 172541 [details]
update math/units to 2.13

maintainer update math/units to 2.13
Comment 1 jharris 2016-07-15 00:11:51 UTC
POLA warning, temperature conversions are now:

You have: tempF (80)
You want: tempC
        26.666667

no longer: 80 degF

for example.
Comment 2 VK freebsd_triage 2016-07-16 18:43:21 UTC
Thanks for the patch. Please confirm the changes pass QA (portlint, poudriere). Removing maintainer-feedback(+) as it was not requested.
Comment 3 VK freebsd_triage 2016-07-16 18:44:18 UTC
Comment on attachment 172541 [details]
update math/units to 2.13

Please set maintainer-approval(+) on patches you submit for ports you maintain.
Comment 4 Dmitry Marakasov freebsd_committer 2016-07-19 14:21:57 UTC
The patch does not apply, has CRLF line endings.
Comment 5 jharris 2016-07-21 18:16:36 UTC
Created attachment 172792 [details]
patch without 0x0d (Carriage Returns) - thanks, Apple OS X!

alias dos2unix "tr -d '\r'"
Comment 6 jharris 2016-07-21 18:29:05 UTC
portlint passes:

  %pwd ; pl -V ; pl .
  /tank/usr/ports/math/units
  portlint version 2.17.3
  looks fine.
Comment 7 jharris 2016-07-22 04:40:33 UTC
Right, is Poudriere a strict requirement?  This is the second failure:

  %sudo poudriere ports -c -m svn+https -p subversion
  [00:00:00] ====>> Creating subversion fs... done
  [00:00:01] ====>> Checking out the ports tree...svn: E000060: Error retrieving REPORT: Operation timed out
  [03:52:29] ====>> Error:  fail
  [03:52:29] ====>> Error while creating ports tree, cleaning up.

Honestly, it shouldn't cleanup after this, but allow "svn co" restarts...
Comment 8 commit-hook freebsd_committer 2016-07-26 15:54:28 UTC
A commit references this bug:

Author: amdmi3
Date: Tue Jul 26 15:54:16 UTC 2016
New revision: 419120
URL: https://svnweb.freebsd.org/changeset/ports/419120

Log:
  - Update to 2.13
  - While here, fix LICENSE

  PR:		211131
  Submitted by:	jharris@widomaker.com (maintainer)

Changes:
  head/math/units/Makefile
  head/math/units/distinfo
Comment 9 commit-hook freebsd_committer 2016-07-26 15:57:30 UTC
A commit references this bug:

Author: amdmi3
Date: Tue Jul 26 15:56:31 UTC 2016
New revision: 419121
URL: https://svnweb.freebsd.org/changeset/ports/419121

Log:
  - Pass maintainership to submitter [1]
  - While here, add LICENSE, switch to USES=localbase, other cosmetic fixes

  PR:		211131 [1]
  Submitted by:	natbsd@instinctive.eu [1]

Changes:
  head/x11-wm/pekwm/Makefile