[MacPorts] #24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- I noticed the ice ports (ice-cpp, ice-java, ice-python, ice-python25, ice- python26) are a bit complex in some places and don't take full advantage of the facilities and features MacPorts makes available. I'd like to propose some changes to simplify these ports. I suggest I attach a patch that makes one change to all these ports, explain why I recommend these changes, and once you or I commit it, I'll attach another patch with another proposal. ---- The first suggestion is to [attachment:branch.diff make and use a ${branch} variable], as described in [wiki:PortfileRecipes#branch PortfileRecipes]. This way you don't have to update the master_sites every time the branch number changes. -- Ticket URL: <http://trac.macports.org/ticket/24904> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Changes (by blair@…): * status: new => assigned Comment: Thanks, applied the branch in r67683. Next please :) -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by ryandesign@…): ---- Next, [attachment:distfiles.diff simplify the distfiles]. I'd remove setting distfiles and remove the distfile name from inside the checksums definition. You're setting distname, and you only have one distfile, so that's enough. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by blair@…): There was a point when there were separate distfiles, such as Google Protocol Buffer support. Fixed in r67707 and r67708. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by ryandesign@…): ---- I'd let MacPorts handle the [attachment:patch.diff patch phase]. In ice- cpp and ice-java you override the patch phase and do the patching manually, but it seems to work fine if you just let MacPorts do it. This patch also includes a refresh of ice-cpp's patchfile so it applies without fuzz. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by blair@…): ice-java fixed in r67800 and ice-cpp fixed in r67802. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by ryandesign@…): ---- Ok, here's a bigger one to [attachment:worksrcpath.diff make more use of worksrcpath and build.dir]. I see a lot of "${workpath}/Ice-${version}" in these portfiles, and this could be simplified to "${worksrcpath}". And there's several places where you're dealing with a specific subdirectory of ${worksrcpath} -- "cpp" in the ice-cpp port, "java" in the java port, "py" in the python ports -- and this could be further abstracted out into the build.dir variable. This will additionally save you from needing to set test.dir or destroot.dir, since both of those variables' default values is ${build.dir}. And this lets you eliminate some other variables (ice-java and cppdir) which end up being the same as ${build.dir}. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by blair@…): Thanks! Applied in r67860. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:7> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Comment(by ryandesign@…): ---- I'd [attachment:variants.diff change the variants] so all the code relating to the variants is inside the variant declarations. Some disagree, but I find this clearer. The Guide in fact advocates the opposite (advocates doing it the way you already do it in these ports), and this used to be required in older versions of MacPorts, but hasn't been for years, and I no longer recommend it. See #18359. -- Ticket URL: <http://trac.macports.org/ticket/24904#comment:8> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24904: ice-cpp, ice-java, ice-python, ice-python25, ice-python26: simplify -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: blair@… Type: enhancement | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: ice-cpp, ice-java, ice-python, ice-python25, ice-python26 -------------------------------------+-------------------------------------- Changes (by snc@…): * cc: snc@… (added) Comment: Along these lines, is it handling the architectures properly, either through build.arch or supported_archs? -- Ticket URL: <https://trac.macports.org/ticket/24904#comment:9> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts