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

correct freescale address #302

Merged
merged 1 commit into from
Mar 16, 2020
Merged

correct freescale address #302

merged 1 commit into from
Mar 16, 2020

Conversation

roumy
Copy link

@roumy roumy commented Mar 16, 2020

change freescale git repo address on rocko branch to make https://wiki.somlabs.com/index.php/Yocto_for_VisionSOM-6ULL/6UL consistent

@otavio
Copy link
Member

otavio commented Mar 16, 2020

You should not bump the revision.

@roumy
Copy link
Author

roumy commented Mar 16, 2020

hi , could you explain me more please

i get that

SRCREV = "f6d0859f9435796f03ae93b70b5f92f4406bc56d

can be removed and latest revision will be automatically get
but i am not the original SRCREV definition , i just correct the stuff to make it work.

@otavio
Copy link
Member

otavio commented Mar 16, 2020

Please keep 951c1363abe95dd75ab3e9447f640d7807240236 as revision, so we don't change it.

@roumy
Copy link
Author

roumy commented Mar 16, 2020

the thing is that original source URI "git.freescale.com/imx/imx-firmware.git"
do no more exist.
It has be migrated to "github.com/NXP/imx-firmware.git" .
may be linked to
nxp-imx/imx-firmware#1

@otavio
Copy link
Member

otavio commented Mar 16, 2020

The SRC_URI is fine, the SRCREV should not change.

@roumy
Copy link
Author

roumy commented Mar 16, 2020

The SRCREV 951c1363abe95dd75ab3e9447f640d7807240236 is not present in github repo.
It cannot work .

@otavio
Copy link
Member

otavio commented Mar 16, 2020

I see.

@otavio otavio merged commit 1d40ea3 into Freescale:rocko Mar 16, 2020
@mattmart3
Copy link

mattmart3 commented Apr 20, 2020

It looks like there is the same issue in the pyro and morty branches. Could you merge it there too?

@otavio
Copy link
Member

otavio commented Apr 20, 2020

Someone need to open specific PRs for those.

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

Successfully merging this pull request may close these issues.

3 participants