Opened 11 years ago

Closed 9 years ago

#834 closed task (fixed)

"mipsel" isn't a usable value for $CLFS_ARCH

Reported by: Andrew Bradford Owned by: Andrew Bradford
Priority: major Milestone: CLFS Embedded 1.0.0
Component: BOOK Version: CLFS Embedded GIT
Keywords: Cc:


In kernel 2.6.38 (and maybe others, haven't checked), a value of "mipsel" isn't valid for the way we use it in the Linux headers installation. The value needs to be "mips" for all variants of mips target triplets.

The offending sed line is:

export CLFS_ARCH=$(echo ${CLFS_TARGET} | sed -e 's/\(mips\)\(64\)\?\(el\)\?-.*/\1\3/')

Could changing this to just be "mips" rather than a possible "mipsel" be bad in other places in the book?

Change History (4)

comment:1 Changed 11 years ago by Andrew Bradford

My environment where I have issues is:


Kernel headers_check will complain like:

Makefile:547: /mnt/clfs/sources/linux- No such file or directory
make: *** No rule to make target `/mnt/clfs/sources/linux-'.  Stop.

comment:2 Changed 11 years ago by Andrew Bradford

In the "Build Variables" section, these two sed lines need to change:

export CLFS_ARCH=$(echo ${CLFS_TARGET} | sed -e 's/\(mips\)\(64\)\?\(el\)\?-.*/\1\3/')
export CLFS_ENDIAN=$(echo ${CLFS_ARCH} | sed -e 's/mipsel/little/' -e 's/mips/big/')

The only places CLFS_ARCH is used for MIPS or WRT builds is in uClibc and the kernel build. Changing CLFS_ARCH to just be "mips" will require a change to the uClibc patches naming convention. Currently the patches naming convention would look like "mipsel.little.blahblah", it'll change to "mips.little.blahblah". That's OK as the "el" was kind of redundant with the "little".

My proposed change is to make the first sed line not care about the "el" and the second sed line should work on the CLFS_TARGET variable rather than CLFS_ARCH.

comment:3 Changed 10 years ago by Andrew Bradford

Owner: changed from clfs-commits@… to Andrew Bradford
Status: newassigned

comment:4 Changed 9 years ago by Andrew Bradford

Resolution: fixed
Status: assignedclosed

This has been fixed, ${CLFS_ARCH} will now be "mips"

Note: See TracTickets for help on using tickets.