1 |
On Sat, Jul 14, 2018 at 08:21:42AM +0200, Ulrich Mueller wrote: |
2 |
> >>>>> On Fri, 13 Jul 2018, Marty E Plummer wrote: |
3 |
> |
4 |
> > In EAPI 7, D, ED, ROOT, EROOT no longer have a trailing slash[1]. This |
5 |
> > makes finding /usr/src/linux not work properly as it currently stands. |
6 |
> |
7 |
> > Use the form "${ROOT%/}/" where apropos in order to unify behavior across |
8 |
> > EAPIs. |
9 |
> |
10 |
> > 1: https://dev.gentoo.org/~ulm/pms/head/pms.html#x1-113001r7 |
11 |
> |
12 |
> Please use https://projects.gentoo.org/pms/7/pms.html if you want the |
13 |
> fragment identifier to be stable (which is not guaranteed for the |
14 |
> version in my home directory). |
15 |
> |
16 |
Ok, will do. |
17 |
> > @@ -554,7 +554,7 @@ get_version() { |
18 |
> > # caught before this if they are. |
19 |
> > if [[ -z ${OUTPUT_DIR} ]] ; then |
20 |
> > # Try to locate a kernel that is most relevant for us. |
21 |
> > - for OUTPUT_DIR in "${SYSROOT}" "${ROOT}" "" ; do |
22 |
> > + for OUTPUT_DIR in "${SYSROOT}" "${ROOT%/}/" "" ; do |
23 |
> |
24 |
> Shouldn't it be "${ROOT%/}" here? Otherwise OUTPUT_DIR may begin with |
25 |
> a double slash. |
26 |
> |
27 |
Ah, good point. I didn't check that well enough, will fix it up. |
28 |
> > OUTPUT_DIR+="/lib/modules/${KV_MAJOR}.${KV_MINOR}.${KV_PATCH}${KV_EXTRA}/build" |
29 |
> > if [[ -e ${OUTPUT_DIR} ]] ; then |
30 |
> > break |