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

dailymotion-cmdline-sample readme minor issues #499

Closed
kilaka opened this issue Sep 18, 2020 · 0 comments
Closed

dailymotion-cmdline-sample readme minor issues #499

kilaka opened this issue Sep 18, 2020 · 0 comments
Assignees
Labels
priority: p3 Desirable enhancement or fix. May not be included in next release. samples Issues that are directly related to samples. type: question Request for information or clarification. Not an issue.

Comments

@kilaka
Copy link

kilaka commented Sep 18, 2020

3 points feedback on the readme:

  1. In the readme exists the following: cd google-oauth-java-client/samples/dailymotion/cmdline-sample:
    instead of dailymotion/cmdline-sample should be dailymotion-cmdline-sample.

  2. After cloning the repo and trying to run mvn compile, as instructed in the readme, getting the following error below.
    This is because master is a SNAPSHOT version - nothing in maven remote repos.
    What I did first build (mvn install) the parent (cloned) folder.

alik@alik:~/dev/google-oauth-java-client/samples/dailymotion-cmdline-sample$ mvn compile
[INFO] Scanning for projects...
[INFO] Inspecting build with total of 1 modules...
[INFO] Installing Nexus Staging features:
[INFO]   ... total of 1 executions of maven-deploy-plugin replaced with nexus-staging-maven-plugin
[INFO] 
[INFO] -----< com.google.oauth-client:dailymotion-simple-cmdline-sample >------
[INFO] Building Example for the DailyMotion API using OAuth2. 1.31.1-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[WARNING] The POM for com.google.oauth-client:google-oauth-client-java6:jar:1.31.1-SNAPSHOT is missing, no dependency information available
[WARNING] The POM for com.google.oauth-client:google-oauth-client-jetty:jar:1.31.1-SNAPSHOT is missing, no dependency information available
[WARNING] The POM for com.google.oauth-client:google-oauth-client:jar:1.31.1-SNAPSHOT is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.639 s
[INFO] Finished at: 2020-09-17T20:09:50Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project dailymotion-simple-cmdline-sample: Could not resolve dependencies for project com.google.oauth-client:dailymotion-simple-cmdline-sample:jar:1.31.1-SNAPSHOT: The following artifacts could not be resolved: com.google.oauth-client:google-oauth-client:jar:1.31.1-SNAPSHOT, com.google.oauth-client:google-oauth-client-java6:jar:1.31.1-SNAPSHOT, com.google.oauth-client:google-oauth-client-jetty:jar:1.31.1-SNAPSHOT: Could not find artifact com.google.oauth-client:google-oauth-client:jar:1.31.1-SNAPSHOT -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
  1. There's a space between exec: and java.
    The command doesn't execute correctly. Deleting the space worked.
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Sep 18, 2020
@kilaka kilaka changed the title dailymotion-cmdline-sample readme minor issues dailymotion-cmdline-sample readme minor issues Sep 18, 2020
kilaka referenced this issue Sep 18, 2020
* Convert assembly readme.html -> README.md and fix links

* Convert and clean dailymotion-cmdline-sample to markdown
@lesv lesv added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Sep 18, 2020
@lesv lesv added the type: question Request for information or clarification. Not an issue. label Sep 18, 2020
@elharo elharo mentioned this issue Sep 18, 2020
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Dec 17, 2020
@elharo elharo added priority: p3 Desirable enhancement or fix. May not be included in next release. and removed priority: p2 Moderately-important priority. Fix may not be included in next release. labels Jan 11, 2021
@elharo elharo closed this as completed Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p3 Desirable enhancement or fix. May not be included in next release. samples Issues that are directly related to samples. type: question Request for information or clarification. Not an issue.
Projects
None yet
Development

No branches or pull requests

5 participants