[Buildroot] [PATCH v1 2/3] aufs: new kernel extension.
Christian Stewart
christian at paral.in
Mon Aug 3 21:23:34 UTC 2015
Hi Yann,
Remove this 'depends on' but enclose the options inside an if...endif
> block.
>
Done here:
https://github.com/paralin/buildroot/commit/8dd9e882fac4a4512d9dcad8fe19da02ff901e45
> + default BR2_PACKAGE_AUFS_4X if BR2_KERNEL_HEADERS_4_0 ||
> BR2_KERNEL_HEADERS_4.1
> > + default BR2_PACKAGE_AUFS_3X if !(BR2_KERNEL_HEADERS_4_0 ||
> BR2_KERNEL_HEADERS_4.1)
>
> See the comment below, about kernel version...
>
As you caught in your next email, these are *defaults*, or guesses. The
user should still verify if the version is correct. The guess is correct a
lot of the time and is helpful to the user because the aufs branches are
often not obviously named.
The help entry goes last (i.e. after the defaults).
>
Fixed.
So, this means we can't build aufs except with an internal toolchain.
>
No, just guesses.
> +define AUFS_STANDALONE_PREPARE_KERNEL
> > + if test -d $(LINUX_DIR)/fs/aufs/; then \
> > + echo "Your kernel already supports AUFS. Not patching."; \
>
> This should be a hard-error.
>
> Also, we want to check that the aufs version indeed matches that of hte
> kernel, and error-out if not.
>
As far as I can tell this is not something the Buildroot system is capable
of doing on its own. It's a complex process to check if the versions match.
The aufs make system will do this on its own, so there is already a hard
error if you have the wrong versions checked out. I vote to just let this
error happen at build time.
Christian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20150803/6ecd8369/attachment-0002.html>
More information about the buildroot
mailing list