Bug 217737

Summary: dns/unbound fails to build with python (swig dependency problem)
Product: Ports & Packages Reporter: geoffroy desvernay <dgeo>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Closed DUPLICATE    
Severity: Affects Some People CC: jaap, pi, w.schwarzenfeld
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
poudriere build log
none
Patch to fis problem
none
Configures the proper version of swig jaap: maintainer-approval+

Description geoffroy desvernay 2017-03-13 10:38:24 UTC
Created attachment 180762 [details]
poudriere build log

my poudriere dns/unbound fails to build since r435651 because of swig dependency update:

devel/swig30 does not provide 'swig' program as devel/swig13 but 'swig3.0'

This only affects builds with PYTHON option (not selected by default).
Comment 1 Jaap Akkerhuis 2017-03-13 15:28:06 UTC
Created attachment 180779 [details]
Patch to fis problem

Yikes. It seems that the last moment suggested update to a newer swig was a bad idea. The patch ai Will attach reverses this decision. Apologies.

        jaap
Comment 2 Walter Schwarzenfeld freebsd_triage 2017-03-15 05:00:03 UTC
I don't think the patch is the right solution.
I think
Makefile
+CONFIGURE_ARGS+=ac_cv_path_SWIG=${LOCALBASE}/bin/swig3.0

and change to:
PYTHON_BUILD_DEPENDS=    swig3.0:devel/swig30
Comment 3 Walter Schwarzenfeld freebsd_triage 2017-03-15 08:53:28 UTC
configure:
16867         # Extract the first word of "swig", so it can be a program name with args.
  16868 set dummy swig; ac_word=$2
  16869 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5
  16870 $as_echo_n "checking for $ac_word... " >&6; }
  16871 if ${ac_cv_path_SWIG+:} false; then :
  16872   $as_echo_n "(cached) " >&6
  16873 else
  16874   case $SWIG in
  16875   [\\/]* | ?:[\\/]*)
  16876   ac_cv_path_SWIG="$SWIG" # Let the user override the test with a path.
  16877   ;;
  16878   *)
  16879   as_save_IFS=$IFS; IFS=$PATH_SEPARATOR


Other ports set also explicit in different ways to swig (e.g graphics/geos)..
Comment 4 Walter Schwarzenfeld freebsd_triage 2017-03-15 08:57:35 UTC
Should be
....set the path to swig...
Comment 5 Jaap Akkerhuis 2017-03-15 09:12:55 UTC
(In reply to w.schwarzenfeld from comment #3)
The patch was just going back to the previous situation.

I'm travelling on the moment but will test you suggestion ASAP. Thanks!
Comment 6 Jaap Akkerhuis 2017-03-23 11:39:50 UTC
Created attachment 181094 [details]
Configures the proper version of swig

Uses the suggestion from w.schwarzenfeld. Thanks!
Comment 7 geoffroy desvernay 2017-04-06 10:46:29 UTC
tested here working. Thanks !

Hopefully a committer will take it quickly :)
Comment 8 Jaap Akkerhuis 2017-04-25 11:25:32 UTC
There is a new version of unbound released, see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=218872, which makes this this moot.

We should close this.
Comment 9 Kurt Jaeger freebsd_committer freebsd_triage 2017-04-29 20:49:37 UTC

*** This bug has been marked as a duplicate of bug 218872 ***