Re: [yocto-autobuilder][PATCH] bin/release_scripts/release.py

["Graydon, Tracy" <tracy.graydon@intel.com>]

On 05/24/16, 4:30 PM, "Randle, William C" <william.c.randle@intel.com>
wrote:

>On Tue, 2016-05-24 at 16:06 -0700, Graydon, Tracy wrote:
>> Get rid of the extraneous DL_BASE line. It's not used and should have
>> been removed when I fixed the eclipse-plugin release path.
>> 
>> Signed-off-by: Graydon, Tracy <tracy.graydon@intel.com>
>> ---
>>  bin/release_scripts/release.py | 1 -
>>  1 file changed, 1 deletion(-)
>> 
>> diff --git a/bin/release_scripts/release.py
>>b/bin/release_scripts/release.py
>> index 2b4d21c..89f68be 100755
>> --- a/bin/release_scripts/release.py
>> +++ b/bin/release_scripts/release.py
>> @@ -351,7 +351,6 @@ if __name__ == '__main__':
>>      VHOSTS = "/srv/www/vhosts"
>>      AB_BASE = os.path.join(VHOSTS,
>> "autobuilder.yoctoproject.org/pub/releases")
>>      DL_DIR = os.path.join(VHOSTS,
>>"downloads.yoctoproject.org/releases")
>> -    DL_BASE = os.path.join(DL_DIR, "/releases/yocto")
>>      ADT_BASE = os.path.join(VHOSTS, "adtrepo.yoctoproject.org")
>>  
>>      # List of the directories we delete from all releases
>
>Since your previous version of this patch has not yet been applied to
>master, I
>would think it would be better to incorporate this as part of a V2 of your
>previous patch, rather than a new patch on top of an as yet unapplied
>patch.

Ah, you are right. I thought it had been pulled in already. I’ll rebase.

>
>    -Bill