Ticket #834 (closed task: fixed)

Opened 7 years ago

Last modified 4 years ago

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

Reported by: bradfa Owned by: bradfa
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

comment:1 Changed 7 years ago by bradfa

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 7 years ago by bradfa

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 6 years ago by bradfa

  • Status changed from new to assigned
  • Owner changed from clfs-commits@… to bradfa

comment:4 Changed 4 years ago by bradfa

  • Status changed from assigned to closed
  • Resolution set to fixed

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

Note: See TracTickets for help on using tickets.