Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add fix for codeaurora.org phaseout #963

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

Open
martab1994 wants to merge 1 commit into main
base: main
Choose a base branch
Loading
from martab1994-portenta-x8-image-build

Conversation

@martab1994
Copy link
Contributor

@martab1994 martab1994 commented Apr 4, 2023

What This PR Changes

  • Add procedure to allow image build also after codeaurora.org phase-out.

Contribution Guidelines

Copy link

gatsby-cloud bot commented Apr 4, 2023

✅ docs-content deploy preview ready

Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting important feedback from @martab1994 and @MaxPayne86 regarding a potential error in the mirrors of the NXP config file

Comment on lines +153 to +158
MIRRORS += " \
git://source.codeaurora.org/external/imx/ git://github.com/nxp-imx/ \n \
https://source.codeaurora.org/external/imx/ https://github.com/nxp-imx/ \n \
http://source.codeaurora.org/external/imx/ http://github.com/nxp-imx/ \n \
gitsm://source.codeaurora.org/external/imx/ gitsm://github.com/nxp-imx/ \n \
```
Copy link
Contributor

@jcarolinares jcarolinares Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@martab1994 and @MaxPayne86 I just wanted to confirm if those "\n" should be there? I think we should remove them?

Copy link
Contributor

@MaxPayne86 MaxPayne86 Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcarolinares I didn't noticed them! So they are needed according to openembedded-core/meta/classes/mirrors.bbclass

@jhansson-ard jhansson-ard added arduino Bugs and fixes added by the Arduino Team pro fix/update A small fix or update labels Apr 20, 2023
@jhansson-ard jhansson-ard added the request Tracking issue label Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaxPayne86 MaxPayne86 MaxPayne86 left review comments

@jcarolinares jcarolinares Awaiting requested review from jcarolinares

Requested changes must be addressed to merge this pull request.

Labels

arduino Bugs and fixes added by the Arduino Team fix/update A small fix or update pro request Tracking issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /