Opened 11 years ago

Closed 3 years ago

#43594 closed defect (fixed)

xmlpatch @0.3.3: "check" should be a library dependency, not just a build dependency

Reported by: cooljeanius (Eric Gallager) Owned by: twic@…
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc:
Port: xmlpatch

Description

I recently had to deactivate my installation of the "check" port to upgrade gstreamer1, because gstreamer1 has the following in its Portfile:

conflicts_build         check

After doing this and installing another port, rev-upgrade then kicked in, and found that xmlpatch had been broken by the deactivation of "check":

--->  Scanning binaries for linking errors
Could not open /opt/local/lib/libcheck.0.dylib: Error opening or reading file (referenced from /opt/local/lib/tests/xml_diff/test-diff)
DEBUG: Marking /opt/local/lib/tests/xml_diff/test-diff as broken
DEBUG: Marking /opt/local/lib/tests/xml_diff/test-diff as broken

Just to confirm, the binary mentioned does actually come from xmlpatch:

$ port provides /opt/local/lib/tests/xml_diff/test-diff
/opt/local/lib/tests/xml_diff/test-diff is provided by: xmlpatch

Running otool -L manually confirms the linkage:

$ otool -L /opt/local/lib/tests/xml_diff/test-diff
/opt/local/lib/tests/xml_diff/test-diff:
	/opt/local/lib/libcheck.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/opt/local/lib/libxmlpatch.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/opt/local/lib/libxmldiff.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/opt/local/lib/libxml2.2.dylib (compatibility version 12.0.0, current version 12.1.0)
	/opt/local/lib/libglib-2.0.0.dylib (compatibility version 3801.0.0, current version 3801.2.0)
	/opt/local/lib/libintl.8.dylib (compatibility version 10.0.0, current version 10.2.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 125.2.11)

The build dependency that xmlpatch has on "check" should be updated to be a library dependency to reflect the fact that this linkage occurs. Either that or you could try linking "check" in statically only, or you could prevent the tests from being installed. Personally I would think that just changing the dependency style from depends_build to depends_lib would be easiest though.

(also, looking at the linkages, you should probably also add a library dependency on gettext, as well, for libintl, although that is not quite as important because so much other stuff depends on gettext that it is much less likely to get deactivated than "check" is, and also nm says that xmlpatch does not actually use any symbols from libintl, even though it links against it...)

Change History (5)

comment:1 in reply to:  description ; Changed 11 years ago by twic@…

Replying to egall@…:

you could prevent the tests from being installed

That's definitely the right fix. I'm not sure how to achieve it, though - i'll have to look into it.

(also, looking at the linkages, you should probably also add a library dependency on gettext, as well, for libintl, although that is not quite as important because so much other stuff depends on gettext that it is much less likely to get deactivated than "check" is, and also nm says that xmlpatch does not actually use any symbols from libintl, even though it links against it...)

This clearly also wants looking into. I suspect that libintl is a transitive dependency of something else; it may or may not actually be needed.

It'd be nice to upgrade to version 0.4.0 as well, once the bugs are fixed.

comment:2 Changed 11 years ago by mf2k (Frank Schima)

Cc: twic@… removed
Owner: changed from macports-tickets@… to twic@…

comment:3 in reply to:  1 Changed 4 years ago by cooljeanius (Eric Gallager)

Replying to twic@…:

It'd be nice to upgrade to version 0.4.0 as well, once the bugs are fixed.

If there's an update available, you might want to check the livecheck regex for it, too, as livecheck currently doesn't notice the update:

$ port -v livecheck xmlpatch
Error: cannot check if xmlpatch was updated (regex didn't match)
$

comment:4 Changed 3 years ago by cooljeanius (Eric Gallager)

I opened the following (draft) PR to do this: https://github.com/macports/macports-ports/pull/11695

Unfortunately, it runs into a build error:

/bin/sh ../libtool --tag=CC   --mode=link /usr/bin/clang -I../src -I/opt/local/include/libxml2 -pipe -Os -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64  -L/opt/local/lib -Wl,-headerpad_max_install_names -Wl,-syslibroot,/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -o xml_patch xml_patch-xml_patch_main.o ../src/libxmlpatch.la -L/opt/local/lib -lxml2 
check_diff.c:243:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_1_to_2", test_12},
                    ^~~~~~~
check_diff.c:244:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_1_to_3", test_13},
                    ^~~~~~~
check_diff.c:245:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_1_to_4", test_14},
                    ^~~~~~~
check_diff.c:246:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_1_to_5", test_15},
                    ^~~~~~~
check_diff.c:247:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_2_to_1", test_21},
                    ^~~~~~~
check_diff.c:248:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_3_to_1", test_31},
                    ^~~~~~~
check_diff.c:249:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_4_to_1", test_41},
                    ^~~~~~~
check_diff.c:250:21: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_5_to_1", test_51},
                    ^~~~~~~
check_diff.c:251:22: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_from_to", test_from_to},
                     ^~~~~~~~~~~~
check_diff.c:252:22: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_to_from", test_to_from},
                     ^~~~~~~~~~~~
check_diff.c:253:23: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_n_0_to_1", test_n01},
                      ^~~~~~~~
check_diff.c:254:23: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_n_0_to_2", test_n02},
                      ^~~~~~~~
check_diff.c:255:23: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_n_2_to_0", test_n10},
                      ^~~~~~~~
check_diff.c:256:23: warning: incompatible pointer types initializing 'void (*)(int)' with an expression of type 'const TTest *' (aka 'const struct TTest *') [-Wincompatible-pointer-types]
    {"test_n_1_to_0", test_n20},
                      ^~~~~~~~
check_diff.c:243:21: error: initializer element is not a compile-time constant
    {"test_1_to_2", test_12},
                    ^~~~~~~
mv -f .deps/xml_diff-xml_diff_main.Tpo .deps/xml_diff-xml_diff_main.Po
/bin/sh ../libtool --tag=CC   --mode=link /usr/bin/clang -I../src -I../diff -I/opt/local/include/libxml2 -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -pipe -Os -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64  -L/opt/local/lib -Wl,-headerpad_max_install_names -Wl,-syslibroot,/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -o xml_diff xml_diff-xml_diff_main.o ../src/libxmlpatch.la ../diff/libxmldiff.la -L/opt/local/lib -lxml2 -L/opt/local/lib -lglib-2.0 -lintl -Wl,-framework -Wl,CoreFoundation 
check_diff.c:262:71: error: too many arguments to function call, expected 6, have 7
    _tcase_add_test (tc, arr_tests[i].fn, arr_tests[i].name, 0, 0, 0, 1);
    ~~~~~~~~~~~~~~~                                                   ^
/opt/local/include/check.h:347:27: note: '_tcase_add_test' declared here
CK_DLL_EXP void CK_EXPORT _tcase_add_test(TCase * tc, const TTest * ttest,
                          ^
14 warnings and 2 errors generated.
make[2]: *** [test_diff-check_diff.o] Error 1
make[2]: *** Waiting for unfinished jobs....
libtool: link: /usr/bin/clang -I../src -I/opt/local/include/libxml2 -pipe -Os -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -Wl,-headerpad_max_install_names -Wl,-syslibroot -Wl,/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -o .libs/xml_patch xml_patch-xml_patch_main.o  -L/opt/local/lib ../src/.libs/libxmlpatch.dylib -lxml2
libtool: link: /usr/bin/clang -I../src -I../diff -I/opt/local/include/libxml2 -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -pipe -Os -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -Wl,-headerpad_max_install_names -Wl,-syslibroot -Wl,/Library/Developer/CommandLineTools/SDKs/MacOSX11.sdk -arch x86_64 -o .libs/xml_diff xml_diff-xml_diff_main.o -Wl,-framework -Wl,CoreFoundation  -L/opt/local/lib ../src/.libs/libxmlpatch.dylib ../diff/.libs/libxmldiff.dylib /opt/local/var/macports/build/_Users_ericgallager_Documents_GitHub_macports-ports_textproc_xmlpatch/xmlpatch/work/libxmlpatch-0.3.3/src/.libs/libxmlpatch.dylib -lxml2 -lglib-2.0 -lintl
make[2]: Leaving directory `/opt/local/var/macports/build/_Users_ericgallager_Documents_GitHub_macports-ports_textproc_xmlpatch/xmlpatch/work/libxmlpatch-0.3.3/tests'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/opt/local/var/macports/build/_Users_ericgallager_Documents_GitHub_macports-ports_textproc_xmlpatch/xmlpatch/work/libxmlpatch-0.3.3'
make: *** [all] Error 2
make: Leaving directory `/opt/local/var/macports/build/_Users_ericgallager_Documents_GitHub_macports-ports_textproc_xmlpatch/xmlpatch/work/libxmlpatch-0.3.3'
Command failed:  cd "/opt/local/var/macports/build/_Users_ericgallager_Documents_GitHub_macports-ports_textproc_xmlpatch/xmlpatch/work/libxmlpatch-0.3.3" && /usr/bin/make -j16 -w all 
Exit code: 2
Error: Failed to build xmlpatch: command execution failed
DEBUG: Error code: CHILDSTATUS 18845 2

Would updating fix this?

Last edited 3 years ago by cooljeanius (Eric Gallager) (previous) (diff)

comment:5 Changed 3 years ago by reneeotten (Renee Otten)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.