From: | Kevin Zhao <kevin.zhaoshuai@×××××.com> | ||
---|---|---|---|
To: | gentoo-catalyst@l.g.o, Kevin Zhao <kevin.zhaoshuai@×××××.com> | ||
Subject: | Re: [gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian. | ||
Date: | Tue, 12 Jan 2016 14:31:20 | ||
Message-Id: | CAKe2hGDnJ7Y48qtVKM1EYf1=D7DUio0mV7grwFZwPBQyq0oU3A@mail.gmail.com | ||
In Reply to: | Re: [gentoo-catalyst][PATCH 2/3] Modify the parameter of ppc64 little-endian, because the ppc64 little-endian has different archtecture and new boot parameters.Don't need the hfs and mapping files as it in Big-endian.Also tweak the code of Big-endian. by Mike Frysinger |
1 | Hi Mike, |
2 | Thanks for your comment.I have send a new patch cover this :-) |
3 | |
4 | 2016-01-09 3:50 GMT+08:00 Mike Frysinger <vapier@g.o>: |
5 | > On 08 Jan 2016 05:04, Kevin Zhao wrote: |
6 | >> + if echo ${clst_subarch} | grep -qE ".+le$" |
7 | > |
8 | > just use a pattern match: |
9 | > if [[ ${clst_subarch} == *le ]] |
10 | > |
11 | >> + then |
12 | >> + flags+=( |
13 | >> + -v -T -l -cache-inodes -chrp-boot |
14 | >> + ) |
15 | > |
16 | > -chrp-boot shows up in both, so factor that out into the common setting ? |
17 | > |
18 | >> + flags+=( |
19 | >> + -chrp-boot -netatalk -hfs -probe -map "${clst_target_path}"/boot/map.hfs \ |
20 | > |
21 | > you don't need to use \ with arrays |
22 | > -mike |