[MacPorts] #24767: New port: thrift
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Apache Thrift http://incubator.apache.org/thrift/ I attach a Portfile -- Ticket URL: <http://trac.macports.org/ticket/24767> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by ryandesign@…): There are a lot of variants. Could we eliminate some of these, for example by making some of them always-on, like maybe the ones that don't have any additional dependencies? Could the python variant use python26 instead of python24? -- Ticket URL: <http://trac.macports.org/ticket/24767#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by naoya.n@…): Replying to [comment:1 ryandesign@…]:
There are a lot of variants. Could we eliminate some of these, for example by making some of them always-on, like maybe the ones that don't have any additional dependencies?
Apache Thrift is supported a lot of language. I think that enough support java, python, ruby, perl, and when all support alyways-on compile time is too long...
Could the python variant use python26 instead of python24? OK.
I attached a new Portfile. -- Ticket URL: <http://trac.macports.org/ticket/24767#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by ryandesign@…): I mean the variants cocoa, html, hs, ocaml, st and xsd. They don't list any dependencies. I compiled the port once without those variants and once with, keeping track of how long it took (using `time`) and how much disk space the installed files occupy (using `du`). With your default variants, it took 3 minutes 40 seconds and 5.2MB of disk; adding in these additional variants, it took 4 minutes 12 seconds and 5.7MB of disk. So it doesn't take much more time or disk space to just remove these variants and build their functionality straight into the port, so I would recommend that since it would simplify the port and remove the need for some user choices. Unless these variants should actually have dependencies and you just haven't listed them. For example, does the ocaml variant require a dependency on the ocaml port? It seems to build without ocaml installed, so maybe not. Some more notes: The port installed /usr/local/lib/libthrift.jar, which violates the mtree. (Everything should be installed somewhere within ${prefix}.) The configure script says "configure: WARNING: unrecognized options: --with-python" You don't need to say "use_bzip2 no"; that's the default. You're missing the long_description keyword (see "port lint"), and your description is rather long. So probably the description you have now should become the long_description and you should write a new shorter string for the description. The perl dependency in the perl variant should be written "path:bin/perl:perl5" as it is in other ports (not "port:perl5"). The php dependency in the php variant should be written "path:bin/php:php5" as it is in other ports (not "port:php5"). I'm confused by the java variant. It adds the configure arg "--enable-gen- java=yes" but "--enable-gen-java" is already in the port's global configure args which is the same thing, isn't it? Should the port's global configure args use "--enable-gen-java=no" instead? -- Ticket URL: <http://trac.macports.org/ticket/24767#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by kenny@…): New portfile supports thrift 0.4.0. I also took care of some of the issues. It now builds with all generators, as the tests fail if to you don't. The libraries to build are still configurable. /usr/local/lib/libthrift.jar is likely still installed in the wrong place, as I didn't look at that issue. I built using -perl -rb, so YMMV. -- Ticket URL: <http://trac.macports.org/ticket/24767#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by naoya.n@…): Thanks for all comments. I think that thrift portfile separate, like thrift-php and thrift-python etc... Is this good thing? -- Ticket URL: <http://trac.macports.org/ticket/24767#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Is any chance to see Thrift as a macport? I think that this library is useful and it would be great to see it as a part of my favorite macosx package manager. Can you please update it to Thrift 0.4.0? (or wait until 0.5.0 - it should be released very soon). -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:8> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Oops my bad. Patchfile #2 is for Thrift 0.4.0 -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:9> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Thrift 0.5.0 is out http://www.apache.org/dyn/closer.cgi?path=/incubator/thrift/0.5.0-incubating... Portfile should be updated. -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:10> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Is any chance to see Thrift in macports? I think it will be useful for one who develops apps with Thrift. -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:11> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Updated Portfile for 0.5.0 -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:12> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Forgot to mention: I also added a variant for Haskell (disabled by default) -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:13> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): The patch generates following warnings. I am not sure what should I do with it. Does anyone has any pointers? dhcp-172-19-29-126:thrift anatol$ sudo port install thrift -java -perl -rb ---> Computing dependencies for thrift ---> Fetching thrift ---> Verifying checksum(s) for thrift ---> Extracting thrift ---> Configuring thrift ---> Building thrift ---> Staging thrift into destroot Warning: violation by /etc Warning: violation by /usr Warning: thrift violates the layout of the ports-filesystems! Warning: Please fix or indicate this misbehavior (if it is intended), it will be an error in future releases! -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:14> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Sorry this is a formatted log {{{ $ sudo port install thrift -java -perl -rb ---> Computing dependencies for thrift ---> Fetching thrift ---> Verifying checksum(s) for thrift ---> Extracting thrift ---> Configuring thrift ---> Building thrift ---> Staging thrift into destroot Warning: violation by /etc Warning: violation by /usr Warning: thrift violates the layout of the ports-filesystems! Warning: Please fix or indicate this misbehavior (if it is intended), it will be an error in future releases! }}} -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:15> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by akitada@…): Replying to [comment:15 anatol.pomozov@…]:
Sorry this is a formatted log
{{{ $ sudo port install thrift -java -perl -rb ---> Computing dependencies for thrift ---> Fetching thrift ---> Verifying checksum(s) for thrift ---> Extracting thrift ---> Configuring thrift ---> Building thrift ---> Staging thrift into destroot Warning: violation by /etc Warning: violation by /usr Warning: thrift violates the layout of the ports-filesystems! Warning: Please fix or indicate this misbehavior (if it is intended), it will be an error in future releases! }}}
It looks like thrift port installs /etc/php.d/thrift_protocol.ini and /usr/lib/python2.6/site-packages/thrift. For Python module, you need to use PY_PREFIX to change the installation path. Others have similar ones. Please consult ./configure -h output and README. -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:16> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): I set following envvars configure.env PHP_PREFIX=${destroot}/lib/php PHP_CONFIG_PREFIX=${destroot}/etc/php.d for PHP and following for Python configure.env PY_PREFIX=${destroot} But I still have the warning -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:17> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by kangas@…): Anatol, I attached an improved version of your "Portfile.050". You cannot have multiple 'configure.env' statements -- they will overwrite one another. Instead, you want this: {{{ configure.env-append JAVA_PREFIX=${prefix}/share/java configure.env-append PERL_PREFIX=${prefix} configure.env-append PHP_PREFIX=${prefix}/lib/php configure.env-append PY_PREFIX=${prefix}/Library/Frameworks/Python.framework/Versions/2.6 }}} -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:19> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift -------------------------------+-------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Keywords: | Port: thrift -------------------------------+-------------------------------------------- Comment(by anatol.pomozov@…): Great thank you. I would really love to see thrift port in macports. Thrift is a great library and many people may find it useful. And BTW Thrift is going to be added to Debian repo soon. -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:20> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24767: New port: thrift --------------------------------+------------------------------------------- Reporter: naoya.n@… | Owner: macports-tickets@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: Port: thrift | --------------------------------+------------------------------------------- Changes (by jmr@…): * status: new => closed * resolution: => fixed Comment: Committed in r78945 with these changes: * removed the distname and worksrcdir lines that were just setting the default values * used apache mirror group * moved configure.env settings into the variants that need them * used $frameworks_dir when setting PY_PREFIX * renamed some variants to match the names used by most other ports * disabled ruby variant for the time being because it installs files straight to their final destination, bypassing the destroot and preventing macports from registering them -- Ticket URL: <https://trac.macports.org/ticket/24767#comment:21> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts