Opened 7 years ago

Closed 6 years ago

#36222 closed defect (invalid)

defect: PortGroup cmake needs depends_skip_archcheck for cmake

Reported by: kimuraw (kimura wataru) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: Cc: cssdev, pixilla (Bradley Giesbrecht), ryandesign (Ryan Schmidt)
Port: PortGroup cmake

Description

I think portgroup cmake needs depends_skip_archcheck.

example:

  1. install cmake
  2. install mysql55 +universal
    • uninstall cmake and install cmake +universal
    • install mysql55

we should be allowed to install universal mysql55 with non-universal cmake.

Index: port1.0/group/cmake-1.0.tcl
===================================================================
--- port1.0/group/cmake-1.0.tcl	(revision 97906)
+++ port1.0/group/cmake-1.0.tcl	(working copy)
@@ -38,6 +38,7 @@
 set cmake_share_module_dir ${prefix}/share/cmake/modules
 
 depends_build-append port:cmake
+depends_skip_archcheck-append cmake
 
 #FIXME: ccache works with cmake on linux
 configure.ccache    no

Change History (9)

comment:1 Changed 7 years ago by jmroot (Joshua Root)

Cc: css@… added

The cmake port already has installs_libs no, which should disable arch checking.

comment:2 Changed 7 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla@… added

Cc Me!

comment:3 Changed 7 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla@… removed

Cc Me!

comment:4 Changed 7 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla@… added

Cc Me!

comment:5 Changed 7 years ago by pixilla (Bradley Giesbrecht)

Something is causing cmake to be upgrade +universal:

$ port -q installed active and name:cmake
  cmake @2.8.10_0 (active)
$ port -q rdeps clucene +universal
  cmake
    libidn
      libiconv
        gperf
      gettext
        ncurses
        expat
    openssl
      zlib
$ sudo port -q upgrade --enforce-variants clucene +universal
--->  Deactivating libiconv @1.14_0
--->  Activating libiconv @1.14_0+universal
--->  Deactivating ncurses @5.9_1
--->  Activating ncurses @5.9_1+universal
--->  Deactivating expat @2.1.0_0
--->  Activating expat @2.1.0_0+universal
--->  Computing dependencies for gettext
--->  Deactivating gettext @0.18.1.1_2
--->  Activating gettext @0.18.1.1_2+universal
--->  Computing dependencies for libidn
--->  Deactivating libidn @1.25_0
--->  Activating libidn @1.25_0+universal
--->  Deactivating zlib @1.2.7_0
--->  Activating zlib @1.2.7_0+universal
--->  Computing dependencies for openssl
--->  Deactivating openssl @1.0.1c_0
--->  Activating openssl @1.0.1c_0+universal
--->  Computing dependencies for cmake
--->  Deactivating cmake @2.8.10_0
--->  Activating cmake @2.8.10_0+universal
...

comment:6 in reply to:  description Changed 6 years ago by ryandesign (Ryan Schmidt)

Cc: ryandesign@… added
Resolution: fixed
Status: newclosed

Replying to kimuraw@…:

I think portgroup cmake needs depends_skip_archcheck.

r103434. I don't know why "installs_libs no" in the cmake port did not make this unnecessary.

comment:7 Changed 6 years ago by jmroot (Joshua Root)

That won't change anything, you explicitly asked to enforce +universal in your upgrade command.

comment:8 Changed 6 years ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: closedreopened

comment:9 Changed 6 years ago by ryandesign (Ryan Schmidt)

Resolution: invalid
Status: reopenedclosed

Yeah... reverted in r104472.

Note: See TracTickets for help on using tickets.