Skip to content

Archives files should not be embedded in archives #1122

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

Closed
d-a-v opened this issue Jan 2, 2021 · 8 comments
Closed

Archives files should not be embedded in archives #1122

d-a-v opened this issue Jan 2, 2021 · 8 comments

Comments

@d-a-v
Copy link
Contributor

d-a-v commented Jan 2, 2021

Bug Report

Current behavior

In some case, library archives generated thanks to dot_a_linkage=true are put into objs.a,
resulting in an archive file embedded into another archive file.

Example in ESP8266Audio CI:

  • 1st .a generated by dot_a_linkage=true (log in CI)
  • 2nd .a generated by arduino-builder (because of a too large number of generated files ?) (log in CI)
    ar cru p/libraries/ESP8266HTTPClient/objs.a p/libraries/ESP8266HTTPClient/ESP8266HTTPClient.a
  • final error in CI

probable source: #961

Expected behavior

Any of the two, or both ?

  • refuse to add .a archive files in AR command line.
  • skip dot_a_linkage=true libraries ?

Environment

  • CLI version (output of arduino-cli version):

arduino-builder shipped with arduino-ide-hourly-build

  • OS and platform:

ubuntu / any

@cmaglie
Copy link
Member

cmaglie commented Jan 26, 2021

Should be fixed by #1140, @d-a-v may you test with the latest nightly?

@d-a-v
Copy link
Contributor Author

d-a-v commented Jan 26, 2021

Is Arduino-IDE-nightly including arduino-builder/cli nightly ?

@per1234
Copy link
Contributor

per1234 commented Jan 27, 2021

@d-a-v no. You can see the version of arduino-builder used in the nightly Arduino CLI build here:
https://github.com/arduino/Arduino/blob/master/build/build.xml#L102

  <property name="ARDUINO-BUILDER-VERSION" value="1.5.5" />

and the version of Arduino CLI used in that version of arduino-builder (confusing because they didn't push a 1.5.5 tag) here:
https://github.com/arduino/arduino-builder/blob/f959e8b154e695eaa3bfaac18f0310a854e228b8/go.mod#L7

	github.com/arduino/arduino-cli v0.0.0-20201210103408-bf7a3194bb63

@cmaglie
Copy link
Member

cmaglie commented Jan 27, 2021

The CLI nightly is here: https://arduino.github.io/arduino-cli/latest/installation/#nightly-builds

As @per1234 said the Arduino IDE nightly does not get the CLI updates automatically, we have to manually update it.
@d-a-v if you need the patch in the Arduino IDE nightly I can push it quickly.

@d-a-v
Copy link
Contributor Author

d-a-v commented Jan 27, 2021

@per1234 @cmaglie Thanks !
Yes, that would allow @earlephilhower to try the fix in CI as described in OP.

@cmaglie
Copy link
Member

cmaglie commented Jan 27, 2021

Yes, that would allow @earlephilhower to try the fix in CI as described in OP.

Ok, the nightly of the Arduino IDE has been updated.

@earlephilhower
Copy link
Contributor

Thank you!

I can confirm the nightly build works successfully with the library that was seeing the issue before.

@per1234 per1234 closed this as completed Jan 28, 2021
@cmaglie
Copy link
Member

cmaglie commented Jan 28, 2021

Thanks for the test!

Closing as fixed...

@rsora rsora reopened this May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants