All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Randle, William C" <william.c.randle@intel.com>
To: "Graydon, Tracy" <tracy.graydon@intel.com>,
	"yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: [yocto-autobuilder][PATCH] bin/release_scripts/release.py
Date: Tue, 24 May 2016 23:30:05 +0000	[thread overview]
Message-ID: <1464132605.2920.54.camel@intel.com> (raw)
In-Reply-To: <1464131193-25397-1-git-send-email-tracy.graydon@intel.com>

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.

    -Bill

  reply	other threads:[~2016-05-24 23:30 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-24 23:06 [yocto-autobuilder][PATCH] bin/release_scripts/release.py Graydon, Tracy
2016-05-24 23:30 ` Randle, William C [this message]
2016-05-24 23:50   ` Graydon, Tracy
  -- strict thread matches above, loose matches on Subject: below --
2016-05-24  2:12 Graydon, Tracy
2016-05-24 13:43 ` Flanagan, Elizabeth
2016-05-24  2:07 Graydon, Tracy
2016-05-24 15:46 ` Randle, William C
2016-05-24 22:58   ` Graydon, Tracy
2016-05-03  7:44 Graydon, Tracy
2016-02-17 11:57 Graydon, Tracy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1464132605.2920.54.camel@intel.com \
    --to=william.c.randle@intel.com \
    --cc=tracy.graydon@intel.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.