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

ESP32 failing to install in CI build #339

Closed
ianfixes opened this issue Dec 28, 2022 · 2 comments · Fixed by #338
Closed

ESP32 failing to install in CI build #339

ianfixes opened this issue Dec 28, 2022 · 2 comments · Fixed by #338
Labels
backend bug Something isn't working

Comments

@ianfixes
Copy link
Collaborator

Issue Summary

via email from @RobTillaart:

Errors in this or that FRAM_I2C build:

Error initializing instance: Loading index file: loading json index file /home/runner/.arduino15/package_esp32_index.json: open /home/runner/.arduino15/package_esp32_index.json: no such file or directory

Arduino or Unit Test Code, Illustrating the Problem

RobTillaart/FRAM_I2C@d2a0622

@hlovdal
Copy link
Contributor

hlovdal commented Dec 29, 2022

This is caused by the missing core update-index step I mentioned in this pull request.

I included some other fixes in that branch earlier before v1.4.0 was made. I have now trimmed down my 2021-02-15_rescue.updates branch to just be fixes related to changes that were made on the original 2021-02-15_rescue

I will create separate pull requests for the other stuff.

@ianfixes
Copy link
Collaborator Author

@hlovdal I'm cherry-picking your commits now, and I will keep the contributions although not in their current form. Commenting here for transparency.

In hlovdal@ad9bc8e I would use the following ruby idioms instead:

      def self.merge_capture_results(args)
-       result = { out: "", err: "", success: true }
-       args.each do |a|
-         result[:out] = result[:out] + a[:out]
-         result[:err] = result[:err] + a[:err]
-         result[:success] = a[:success] unless a[:success]
-       end
-       result
+       {
+          out: args.map { |a| a[:out] }.join(""),
+          err: args.map { |a| a[:err] }.join(""),
+          success: args.all? { |a| a[:success] }
+       }
      end

This code for merge_capture_results is referenced in hlovdal@50cb640, but I notice an issue here as well: the install command is not run conditionally on whether update-index succeeds.

I will make the appropriate changes.

And I want to stress that I am very grateful for your help in both diagnosing and patching this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants