[ 
https://issues.apache.org/jira/browse/THRIFT-71?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jared Bunting updated THRIFT-71:
--------------------------------

    Attachment: THRIFT-71_v9.patch

I ran into two small problems with the patch so I'm attaching my modifications.

1.  I know that this patch is intended for the source tree, but when applied to 
the source tarball, debian/rules doesn't work due to invoking bootstrap.sh.  
So, I've changed the invocation of bootstrap.sh to only occur if bootstrap.sh 
is present.

2.  Since I don't have php installed on my system (and it's not a build-dep), 
configure failed looking for "php-config".  I have modified the invocation of 
configure to pass --without-php --without-php-extension.  An alternative would 
be to add php as a build dependency, but that didn't seem appropriate since we 
aren't building a php package.  It is however, entirely possible that I have 
misunderstood the role of php in the build, and if that is the case a build 
dependency would be the correct way to fix this.

> Debian packaging for thrift
> ---------------------------
>
>                 Key: THRIFT-71
>                 URL: https://issues.apache.org/jira/browse/THRIFT-71
>             Project: Thrift
>          Issue Type: Improvement
>         Environment: debian stable (etch); probably will work on lenny, but 
> untested
>            Reporter: Eric Anderson
>            Priority: Minor
>         Attachments: debian-packaging.patch, thrift-71.patch, 
> thrift-71_v2.patch, thrift-71_v3.patch, thrift-71_v4.patch, 
> thrift-71_v5.patch, thrift-71_v6.patch, thrift-71_v7.patch, 
> THRIFT-71_v8_0.5.0.patch, THRIFT-71_v9.patch
>
>
> We're using thrift on Debian, so I wanted to have a package for easier
> installation by other users.
> Attached is a patch to add packaging for debian to the
> thrift-20080411p1 release.  It includes two bug fixes to make the
> tests work reliably: 1) adding /usr/share/java/commons-lang.jar to the
> search path for the java tests, and 2) sleeping 1 second after
> starting the server for the python tests.  If the patch is accepted,
> the maintainer should probably be set to the thrift mailing list,
> rather than the example mailing list address I used.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to