Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Incorrect processing of adding of dependency. #23

Closed
jolly-fellow opened this issue Mar 23, 2023 · 2 comments · Fixed by #47
Closed

Incorrect processing of adding of dependency. #23

jolly-fellow opened this issue Mar 23, 2023 · 2 comments · Fixed by #47
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation 👍 lgtm
Milestone

Comments

@jolly-fellow
Copy link
Contributor

jolly-fellow commented Mar 23, 2023

As a user who doesn't know the antler-proj internals I would be confused of this error message:

root@c1742fb0fdb3:/app# antler-proj add /host/home/alex/dev/projects/AntelopeIO/Clion_DUNE/test dep test_app test_dep "https://github.com/larryk85/sample_contract"
[error_log] :: In this version of antler-proj only github shorthands are supported. Generalized git repos and archives will be supported in the next version.
[error_log] :: Error denpendency: test_dep is invalid.
[error_log] :: Dependency: test_dep is invalid.
[error_log] :: failed to add dependency

I want to add a dependency from github, I declared full path to the repository on github, I don't understand why it is wrong and why the program can't accept this repository. I mean that as a user I don't understand why "https://github.com/larryk85/sample_contract" is wrong but "larryk85/sample_contract" is OK.

After successful adding (according to the message) of the dependency I have following message:

root@c1742fb0fdb3:/app# antler-proj add /host/home/alex/dev/projects/AntelopeIO/Clion_DUNE/test dep test_app test_dep "larryk85/sample_contract"
sh: 1: gh: not found
^^^^^^^^^^^^^^^^ As a user I don't understand what is this
[info_log] ::
^^^^^^^^^ In looks like here should be some message but it was not printed because of bug
Object name (to update): test_app
Dependency name: test_dep
Dependency location: larryk85/sample_contract
tag/commit hash:
release version:
SHA256 hash:

And actually the dependency was not added. The project file was not changed:

project: test_proj
version: 0.0.1
apps:
  - name: test_app
    lang: CXX
    compile_options: ""
    link_options: ""
libs:
  - name: test_lib
    lang: CXX
    compile_options: compiler_opts
    link_options: linker_opts

After try to add dependency to the lib antler-proj returns the following error message:

root@c1742fb0fdb3:/app# antler-proj add /host/home/alex/dev/projects/AntelopeIO/Clion_DUNE/test dep test_lib test_dep "larryk85/sample_contract" 0.0.1
[error_log] :: Object test_lib does not exist in project.
[info_log] :: Wrote project manifest to /host/home/alex/dev/projects/AntelopeIO/Clion_DUNE/test.

@jolly-fellow jolly-fellow added the bug Something isn't working label Mar 23, 2023
@stephenpdeos stephenpdeos added triage documentation Improvements or additions to documentation 👍 lgtm and removed triage labels Apr 7, 2023
@stephenpdeos stephenpdeos added this to the CDT 4.0.0-rc1 milestone Apr 10, 2023
@stephenpdeos
Copy link
Member

Possible this may fall to RC2

@mikelik mikelik self-assigned this Apr 21, 2023
@mikelik
Copy link
Contributor

mikelik commented Apr 21, 2023

To structure this issue:

  1. antler-proj add dep should handle full github URL i.e. https://github.com/larryk85/sample_contract
  2. There should be no error sh: 1: gh: not found (dependency should be simply added)
  3. antler-proj add dep should add dependency to a library

Regarding 3:
With above project.yml and command I cannot reproduce it. Currently it works as expected.

@mikelik mikelik moved this from Todo to In Progress in Team Backlog Apr 21, 2023
@mikelik mikelik linked a pull request Apr 24, 2023 that will close this issue
@mikelik mikelik moved this from In Progress to Awaiting Review in Team Backlog Apr 25, 2023
@github-project-automation github-project-automation bot moved this from Awaiting Review to Done in Team Backlog Apr 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working documentation Improvements or additions to documentation 👍 lgtm
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants