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

Wrong header name #1459

Closed
FStefanni opened this issue Sep 20, 2021 · 5 comments
Closed

Wrong header name #1459

FStefanni opened this issue Sep 20, 2021 · 5 comments
Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project

Comments

@FStefanni
Copy link

Bug Report

Current behavior

arduino-cli lib install --git-url https://github.com/FStefanni/arduino-sht.git

The command fails with the following log:

Error initializing instance: loading platform release esp32:[email protected]: loading boards: skipping loading of boards esp32:esp32:atmegazero_esp32s2, esp32:esp32:deneyapmini: malformed custom board options
--git-url and --zip-path flags allow installing untrusted files, use it at your own risk.
Enumerating objects: 68, done.
Counting objects: 100% (68/68), done.
Compressing objects: 100% (44/44), done.
Total 68 (delta 32), reused 49 (delta 19), pack-reused 0
Error installing Git Library: Library install failed: library is not valid: missing header file "arduino-sh.h"

Expected behavior

The searched header file should be arduino-sht.h and not arduino-sh.h.

Environment

  • CLI version (output of arduino-cli version): arduino-cli alpha Version: 0.19.0 Commit: 56419ec Date: 2021-09-02T14:47:35Z
  • OS and platform: Debian testing (bookworm)

Additional context

  • The library is a clone of arduino-sht, on which I am working to create a pr.
  • Please notice that I have added the arduino-sht.h file.
  • If I add the wrong file arduino-sh.h, the library is successfully installed.

The content of library.properties is:

name=arduino-sht
version=1.2.0
author=Johannes Winkelmann, Andreas Brauchli
maintainer=Johannes Winkelmann <[email protected]>
sentence=Support for Sensirion's humidity and temperature sensors.
paragraph=Supported sensors: SHTC1, SHTC3, SHTW1, SHTW2, SHT3x-DIS (I2C), SHT3x-ARP, SHT4x
category=Sensors
url=https://developer.sensirion.com
architectures=*
@per1234
Copy link
Contributor

per1234 commented Sep 20, 2021

Hi @FStefanni. Thanks for reporting this.

It's related to #1306 in that this check shouldn't be done at all. However, the "sht" vs. "sh" issue is an additional bug that might indicate there is a separate issue that will persist even after #1306 is fixed.

@FStefanni
Copy link
Author

Hi,

yes this is surely related, even if not exactly the same.
I hope this will be fixed asap, since it is very blocking.

Regards.

@FStefanni FStefanni changed the title Wron header name Wrong header name Sep 20, 2021
@per1234
Copy link
Contributor

per1234 commented Sep 20, 2021

I don't see any reason it should be blocking. arduino-cli lib install --git-url is nothing more than a trivial convenience command. Just use git clone to install the library instead. If you don't have Git installed on your computer then download the zip file of the repository from GitHub and use arduino-cli lib install --zip-path to install it.

@FStefanni
Copy link
Author

Hi,

it is blocking for my workflow (of course not in general), because I have a set of scripts using arduino-cli lib install --git-url to automatically install/create the environment to run a set of compiling tasks.

With this bug, I have to change all the scripts, or to install manually the lib on the hosts.

Regards.

@silvanocerza
Copy link
Contributor

Fixed by #1462.

@per1234 per1234 added the type: imperfection Perceived defect in any part of project label Sep 24, 2021
@per1234 per1234 added the conclusion: resolved Issue was resolved label Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

No branches or pull requests

3 participants