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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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 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 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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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