[MacPorts] #67656: svndigest 0.11

MacPorts noreply at macports.org
Tue Jun 20 04:28:28 UTC 2023


#67656: svndigest 0.11
-------------------------+---------------------------------
  Reporter:  peter31042  |      Owner:  ryandesign
      Type:  update      |     Status:  accepted
  Priority:  Normal      |  Milestone:
 Component:  ports       |    Version:
Resolution:              |   Keywords:  haspatch maintainer
      Port:  svndigest   |
-------------------------+---------------------------------

Comment (by ryandesign):

 I see that help2man is used:

 {{{
 checking whether to use help2man... yes
 }}}

 so I'll add a build dependency on the help2man port. macOS includes
 help2man but using the MacPorts version reduces surprises by ensuring
 we're using the same version on every macOS version.

 The build fails when gsl is not installed:

 {{{
 ./yat/utility.h:38:10: fatal error: 'gsl/gsl_math.h' file not found
 #include <gsl/gsl_math.h>
          ^~~~~~~~~~~~~~~~
 }}}

 so I can add that build dependency too. Or perhaps even better, can that
 gsl `#include` line be removed? svndigest builds fine and tests pass
 without gsl installed when that line is removed.

 {{{
 ============================================================================
 Testsuite summary for svndigest 0.11
 ============================================================================
 # TOTAL: 41
 # PASS:  14
 # SKIP:  27
 # XFAIL: 0
 # FAIL:  0
 # XPASS: 0
 # ERROR: 0
 ============================================================================
 }}}

 I see that the gsl `#include` was already
 [https://dev.thep.lu.se/yat/changeset/3938/trunk/yat/utility/utility.h
 removed from upstream yat 3 years ago] but svndigest bundles a 10.5-year-
 old yat. That commit also removed the use of `M_LN2` which is defined by
 gsl/gsl_math.h (if not already defined):

 {{{
 #ifndef M_LN2
 #define M_LN2      0.69314718055994530941723212146      /* ln(2) */
 #endif
 }}}

 My system's math.h already defines `M_LN2` on macOS 12 (and I checked a
 Mac OS X 10.6 system and it appears to as well) but if this is the only
 thing from gsl/gsl_math.h that was being used in yat and if support for
 systems where math.h doesn't define `M_LN2` is important maybe that
 `#ifndef` block can be used in yat/utility.h instead of including
 gsl/gsl_math.h.

 Would updating svndigest to use a newer yat be useful? There have been
 dozens of releases of yat since the one included in svndigest.

-- 
Ticket URL: <https://trac.macports.org/ticket/67656#comment:2>
MacPorts <https://www.macports.org/>
Ports system for macOS


More information about the macports-tickets mailing list