search

u-blox / ubxlib

Portable C libraries which provide APIs to build applications with u-blox products and services. Delivered as add-on to existing microcontroller and RTOS SDKs.
Apache License 2.0
287 stars 82 forks source link

unable to retrieve ubxllib using west: FATAL ERROR: can't import from project ubxlib #173

Closed mabroens closed 7 months ago

mabroens commented 7 months ago

Hi,

I am trying to add ubxlib to my west repo following the steps here https://github.com/u-blox/ubxlib/tree/master/port/platform/zephyr :

  remotes:
    - name: u-blox
      url-base: git@github.com:u-blox

  projects:
    - name: ubxlib
      remote: u-blox
      path: ubxlib
      revision: v1.2.0
      import: true
      clone-depth: 1

but whenever i run west update i get an error:

FATAL ERROR: can't import from project ubxlib
  Expected to import from west.yml at revision 86c97d39dd43c2c5f1ce08a5a4d1619927b8f832
  Hint: possible manifest file fixes for ubxlib:
          - set "revision:" to a git ref with this file at URL https://github.com/u-blox/ubxlib
          - remove the "import:"
        Use "west -vvv update" to debug.

BTW i had to chage the url to:

    - name: u-blox
      url-base: https://github.com/u-blox

for it to work

does anyone have any suggestions?

Br Martijn

mabroens commented 7 months ago

Issue resolved. after changing to the following it worked.

    - name: u-blox
      url-base: https://github.com/u-blox   
    - name: ubxlib
      remote: u-blox
      path: ubxlib
      import: true
      clone-depth: 1
RobMeades commented 7 months ago

Sorry for this, good to hear it's working: does this mean we need to remove the remotes: and projects: lines from our west.yml recommendation; we don't generally use Zephyr internally so am unsure of the accepted syntax?

mabroens commented 7 months ago

Hi Rob,

not sure for me to get it running i had to remove revision: v1.2.0 and had to change the url to: url-base: https://github.com/u-blox

but removing the revision i think is a bad idea. but i have no idea yet how to get the latest version.

after that i was able to get the module.

RobMeades commented 7 months ago

but removing the revision i think is a bad idea

We don't really do revisions: we have release tags but they only mean "the feature we said would be present at this tag is present from this point onwards"; otherwise every commit is tested in the same way and is no different from any other.

mabroens commented 7 months ago

i just tried with revision set for 1.3.1 and that workes as well. no clue why 1.2.0 didnot.

RobMeades commented 7 months ago

Instructions now update, removing revision as it is somewhat limiting.