Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gherkin 6/7 executable problems #589

Closed
aslakhellesoy opened this issue Mar 29, 2019 · 14 comments
Closed

Gherkin 6/7 executable problems #589

aslakhellesoy opened this issue Mar 29, 2019 · 14 comments

Comments

@aslakhellesoy
Copy link
Contributor

Gherkin 6 bundles go executables inside packages (gems, npm modules, jars), and we're not yet sure whether or not this causes problems for users.

If it does, please comment on this issue with as much detail as you can:

  • Operating system
  • Platform/programming language and version
  • Whether you have problems installing/downloading Cucumber
  • Whether you have problems running Cucumber
  • Any error messages you are seeing
@aslakhellesoy aslakhellesoy added library: gherkin 🧷 pinned Tells Stalebot not to close this issue labels Mar 29, 2019
@alfonso-presa
Copy link

alfonso-presa commented Apr 1, 2019

Looks like this might be related:

        <dependency>
            <groupId>io.cucumber</groupId>
            <artifactId>gherkin</artifactId>
            <version>6.0.17</version>
        </dependency>

@aslakhellesoy
Copy link
Contributor Author

Thanks @alfonso-presa - I will look i to this.

@alfonso-presa
Copy link

Thank you @aslakhellesoy. Let me know if you need any further information to get context of the error. I'm using the gherkin parser in a java project to extract meta-data from the gherkin files in order to perform a later distributed execution (in separate processes and machines) using cucumber. I guess this is not the regular use case for this tool so I might have hitten an edge case.

I'm doing researches myself and will let you know if I get any insights or findings. I'll also be happy to PR if I can help.

Keep it up!

@alfonso-presa
Copy link

Oh, @aslakhellesoy. BTW, I'm sorry to be fussy, but there's a typo in the issue title that might make difficult for people to find it as reference for the matter.... You might like to fix it just in case

@aslakhellesoy aslakhellesoy changed the title Gherkin 6 ecxecutable problems Gherkin 6 executable problems Apr 2, 2019
@alfonso-presa
Copy link

Looks like the io.cucumber:gherkin:6.0.17:jar is missing the folder called gherkin-go, that I can find in for example the io.cucumber:gherkin:6.0.10:har release. My whole process is working against 6.0.10 but it's not with 6.0.17. I've checked and it's missing also in 6:0.16 but it's present in 6.0.14 (I'm unable to download 6.0.15 for some reason so I cannot check there).

@aslakhellesoy
Copy link
Contributor Author

Thanks for investigating that @alfonso-presa - that means there is a problem with our build/release process.

@aslakhellesoy
Copy link
Contributor Author

Looking at the release build:

I think the fix is to:

  • Add skip_cleanup: true to the Travis deploy job config
  • Make maven's jar target fail if there are no executables

@alfonso-presa
Copy link

May be using <useStrictFiltering>true</useStrictFiltering> in the assembly.xml file here could help: https://github.com/cucumber/cucumber/blob/master/gherkin/java/assembly.xml#L21

About the skip_cleanup: true I guess it would do the job :-). I'm not sure about how the whole pipeline is supposed to work....

@aslakhellesoy
Copy link
Contributor Author

May be using <useStrictFiltering>true</useStrictFiltering> ...

Thanks for the pointer - that's very helpful

We use skip_cleanup: true in gherkin-ruby and gherkin-javascript so I'm pretty sure that's the problem.

@alfonso-presa
Copy link

Perfect then. Let me know if you want me to PR, although I guess it will be way faster for you to patch it :-).

@aslakhellesoy
Copy link
Contributor Author

Leave it to me - I'll make similar changes to other modules too in the same commit. Thanks!

@aslakhellesoy
Copy link
Contributor Author

@alfonso-presa the latest 6.0.18-SNAPSHOT has a fix for this - see https://docs.cucumber.io/installation/java/ for details on how to access it.

I'm reopening this issue since it's a more general placeholder for other issues related to Gherkin 6.

@aslakhellesoy aslakhellesoy reopened this Apr 16, 2019
@alfonso-presa
Copy link

@aslakhellesoy great!! Thank you very much! Keep it up!

@luke-hill luke-hill changed the title Gherkin 6 executable problems Gherkin 6/7 executable problems Aug 19, 2019
@mpkorstanje mpkorstanje removed the 🧷 pinned Tells Stalebot not to close this issue label May 19, 2020
@mpkorstanje
Copy link
Contributor

Gherkin 6 bundles go executables inside packages (gems, npm modules, jars), and we're not yet sure whether or not this causes problems for users.

The latest versions of Gherkin no longer bundle executables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants