[ 
https://issues.apache.org/jira/browse/THRIFT-2922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14598728#comment-14598728
 ] 

Jake Farrell commented on THRIFT-2922:
--------------------------------------

[~jensg] your patch was close, needed to check for the TRIAL apps existence and 
then either enable or disable has_python since its required for the py.twisted 
unit tests. With new patch get 

{code} 
/usr/bin/trial ./test_suite.py
test_suite
  ThriftTestCase
    testByte ...                                                           [OK]
    testDouble ...                                                         [OK]
    testException ...                                                      [OK]
    testI32 ...                                                            [OK]
    testI64 ...                                                            [OK]
    testOneway ...                                                         [OK]
    testString ...                                                         [OK]
    testStruct ...                                                         [OK]
    testVoid ...                                                           [OK]

-------------------------------------------------------------------------------
Ran 9 tests in 1.048s

PASSED (successes=9)
{code} 

> $TRIAL is used with Python tests but not tested accordingly
> -----------------------------------------------------------
>
>                 Key: THRIFT-2922
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2922
>             Project: Thrift
>          Issue Type: Bug
>          Components: Python - Library
>            Reporter: Jens Geyer
>            Priority: Minor
>             Fix For: 0.9.3
>
>         Attachments: 
> THRIFT-2922_TRIAL-is-required-for-py.Twisted-tests-but-not-fully.patch
>
>
> I did not install trial at first, not knowing that I'm going to need it. The 
> twisted test does not check whether $TRIAL actually has been found by 
> configure, it simply tries to call an empty command then, leading to a funny 
> error message about the first argument not being executable.
> The workaround is to simply install trial.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to