[LEDE-DEV] [RFC 01/13] procd: prepare NAND sysupgrade for making upgraded dynamically linked

Matthias Schiffer mschiffer at universe-factory.net
Mon Apr 24 13:20:38 PDT 2017


On 04/24/2017 10:05 PM, Philip Prindeville wrote:
> Inline…
> 
>> On Apr 23, 2017, at 6:06 PM, Matthias Schiffer <mschiffer at universe-factory.net> wrote:
>>
>> Use install_bin to copy upgraded with all dependencies. The old name
>> /tmp/upgraded is temporarily retained as a symlink to avoid breaking
>> things.
>>
>> Signed-off-by: Matthias Schiffer <mschiffer at universe-factory.net>
>> ---
>> package/system/procd/files/nand.sh | 9 +++++----
>> 1 file changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/package/system/procd/files/nand.sh b/package/system/procd/files/nand.sh
>> index 01dba61644..9c831df3b4 100644
>> --- a/package/system/procd/files/nand.sh
>> +++ b/package/system/procd/files/nand.sh
>> @@ -194,7 +194,7 @@ nand_upgrade_prepare_ubi() {
>>
>> nand_do_upgrade_success() {
>> 	local conf_tar="/tmp/sysupgrade.tgz"
>> -	
>> +
>> 	sync
>> 	[ -f "$conf_tar" ] && nand_restore_config "$conf_tar"
>> 	echo "sysupgrade successful"
>> @@ -231,7 +231,7 @@ nand_upgrade_ubifs() {
>> 	local rootfs_length=`(cat $1 | wc -c) 2> /dev/null`
>>
>> 	nand_upgrade_prepare_ubi "$rootfs_length" "ubifs" "0" "0"
>> -	
>> +
> 
> 
> Please avoid whitespace-only changes.

Well, this is trailing whitespace, which every sensible editor should strip
automatically (or at least warn about it). In my opinion, this change
should be done, as a lot of developers will stumble over it, and would have
to reconfigure their editors or exclude the changes from their commits...


> 
> 
>> 	local ubidev="$( nand_find_ubi "$CI_UBIPART" )"
>> 	local root_ubivol="$(nand_find_volume $ubidev rootfs)"
>> 	ubiupdatevol /dev/$root_ubivol -s $rootfs_length $1
>> @@ -333,7 +333,7 @@ nand_upgrade_stage1() {
>> 		[ "$SAVE_CONFIG" != 1 -a -f "$CONF_TAR" ] &&
>> 			rm $CONF_TAR
>>
>> -		ubus call system nandupgrade "{\"path\": \"$path\" }"
>> +		ubus call system nandupgrade "{\"prefix\": \"$RAM_ROOT\", \"path\": \"$path\" }"
>> 		exit 0
>> 	}
>> }
>> @@ -370,6 +370,7 @@ nand_do_platform_check() {
>> # $(1): file to be used for upgrade
>> nand_do_upgrade() {
>> 	echo -n $1 > /tmp/sysupgrade-nand-path
>> -	cp /sbin/upgraded /tmp/
>> +	install_bin /sbin/upgraded
>> +	ln -s "$RAM_ROOT"/sbin/upgraded /tmp/upgraded
>> 	nand_upgrade_stage1
>> }
>> -- 
>> 2.12.2
>>
>>
>> _______________________________________________
>> Lede-dev mailing list
>> Lede-dev at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/lede-dev
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/lede-dev/attachments/20170424/bd801398/attachment.sig>


More information about the Lede-dev mailing list