Skip to content

mips: Upgrade toolchain to version 2017.10-08#47

Closed
jia200x wants to merge 4 commits into
RIOT-OS:masterfrom
jia200x:pr/mips_dependencies
Closed

mips: Upgrade toolchain to version 2017.10-08#47
jia200x wants to merge 4 commits into
RIOT-OS:masterfrom
jia200x:pr/mips_dependencies

Conversation

@jia200x

@jia200x jia200x commented Aug 29, 2018 •
edited by cladmi
Loading

Copy link
Copy Markdown
Member

This PR fixes the MIPS toolchain link, as stated in #46

Issues/PRs references

Tracking the needed changes to update the toolchain: RIOT-OS/RIOT#10009

@jia200x jia200x mentioned this pull request Aug 29, 2018
Merged
@kaspar030

kaspar030 commented Aug 29, 2018

Copy link
Copy Markdown
Contributor

Can someone test if the toolchain works as expected?

@cladmi

cladmi commented Sep 5, 2018

Copy link
Copy Markdown
Contributor

I will test.

cladmi
cladmi reviewed Sep 5, 2018
View reviewed changes
Comment thread Dockerfile Outdated
wget -q http://codescape.mips.com/components/toolchain/2017.10-07/Codescape.GNU.Tools.Package.2017.10-07.for.MIPS.MTI.Bare.Metal.CentOS-5.x86_64.tar.gz -O- \
| tar -C /opt -xz

ENV PATH $PATH:/opt/mips-mti-elf/2016.05-03/bin

@cladmi cladmi Sep 5, 2018

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This one also needs to be adapted /opt/mips-img-elf/2017.10-08/bin

@cladmi

cladmi commented Sep 5, 2018

Copy link
Copy Markdown
Contributor

FIrst the name changed from mips-mti-elf-gcc to mips-img-elf-gcc.

It also does not compile examples/hello-world with all 3 mips boards.

error '_mips_handle_exception' redeclared with conflicting 'nomips16' attributes

So there is some work to do.

@cladmi

cladmi commented Sep 5, 2018

Copy link
Copy Markdown
Contributor

If someone still has the old archive, it could be hosted on the riot server until the support is updated.

HendrikVE added a commit to riot-appstore/rapstore-django that referenced this pull request Sep 20, 2018
@HendrikVE
fix broken link as shown here: RIOT-OS/riotdocker#47
c1f2f3c
@kaspar030

kaspar030 commented Sep 20, 2018

Copy link
Copy Markdown
Contributor

We do have the extracted toolchain in the deployed containers. While not preferable at all, we could tar.gz that up and use until someone finds the original file or we fix the compilation errors.

@jnohlgard jnohlgard mentioned this pull request Sep 21, 2018
Closed
@jnohlgard

jnohlgard commented Sep 21, 2018

Copy link
Copy Markdown
Member

I updated the PR with the URL to the latest release and updated the PATH variable.
See RIOT-OS/RIOT#10009 for a fix for the build issue

@cladmi

cladmi commented Sep 21, 2018

Copy link
Copy Markdown
Contributor

Found the issue I had while testing, I downloaded the 'IMG' archive instead of the 'MTI' one.

@gschorcht gschorcht mentioned this pull request Sep 28, 2018
Merged
@jnohlgard jnohlgard mentioned this pull request Oct 4, 2018
Merged
@jnohlgard jnohlgard changed the title mips: fix broken toolchain link mips: Update toolchain to version 2017.10-08 Oct 4, 2018
@jnohlgard jnohlgard changed the title mips: Update toolchain to version 2017.10-08 mips: Upgrade toolchain to version 2017.10-08 Oct 8, 2018
@kaspar030 kaspar030 mentioned this pull request Jul 18, 2019
Merged
@kaspar030

kaspar030 commented Jul 18, 2019

Copy link
Copy Markdown
Contributor

Obsoleted by #76.

@kaspar030 kaspar030 closed this Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@cladmi cladmi cladmi left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jia200x @kaspar030 @cladmi @jnohlgard