microsoft / Recognizers-Text

Microsoft.Recognizers.Text provides recognition and resolution of numbers, units, date/time, etc. in multiple languages (ZH, EN, FR, ES, PT, DE, IT, TR, HI, NL. Partial support for JA, KO, AR, SV). Packages available at: https://www.nuget.org/profiles/Recognizers.Text, https://www.npmjs.com/~recognizers.text
MIT License
1.68k stars 430 forks source link

[Java] - README instructions are insufficient to run the sample code #3141

Open matthew-carroll opened 11 months ago

matthew-carroll commented 11 months ago

Describe the bug The Java sample README says to run mvn exec:java from within the simple-console directory. However, for me, that was insufficient. Maven complained about not being able to find the main file - I got a class not found exception at the terminal. I couldn't run the sample.

To Reproduce Steps to reproduce the behavior:

  1. Clone the repo
  2. Navigate to the simple-console Java directory
  3. Run mvn exec:java

Expected behavior The sample app runs.

Actual behavior I get a class not found exception for the Sample class.

Resolution Running mvn clean install from within the simple-console directory solved the problem. HOWEVER, to get mvn clean install to run, I had to copy the two lint configuration XML files from the top-level Java directory down into the simple-console directory. Without copying those files, mvn clean install was failing on the lint lifecycle.

The README for the sample app should include all necessary steps to build and run the sample app, and that build process shouldn't require copying any files.

Platform (please complete the following information):