You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2014/12/30 00:06:13 UTC

[jira] [Commented] (THRIFT-2920) Erlang Thrift test uses wrong IDL file

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

Hudson commented on THRIFT-2920:
--------------------------------

FAILURE: Integrated in Thrift #1419 (See [https://builds.apache.org/job/Thrift/1419/])
THRIFT-2920 Erlang Thrift test uses wrong IDL file (jensg: rev 326cf0d0fbb6ef79358500b17fa0a9327d2e6df9)
* test/erl/Makefile.am


> Erlang Thrift test uses wrong IDL file
> --------------------------------------
>
>                 Key: THRIFT-2920
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2920
>             Project: Thrift
>          Issue Type: Bug
>          Components: Erlang - Library, Test Suite
>            Reporter: Jens Geyer
>            Assignee: Jens Geyer
>             Fix For: 0.9.3
>
>         Attachments: THRIFT-2920-Erlang-Thrift-test-uses-wrong-IDL-file.patch
>
>
> Erlang test executes the following statement:
> {code:title=test/erl/Makefile.am}
> .generated: $(THRIFT_FILES)
> 	for f in $(THRIFT_FILES) ; do \
> 	  $(THRIFT) --gen erl $$f ; \
> 	done ; \
> 	$(THRIFT) --gen erl:legacynames LegacyNames.thrift
> 	touch .generated
> {code}
> Because ThriftTest.thrift and DocTest.thrift both expose a {{service ThriftTest}} whose contents do *not* match, depending on the order of how the system sees the files, the code generated out of DocTest.thrift may overwrite the previously generated code from ThriftTest.thrift.



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