All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Katu Txakur <katutxakurra@gmail.com>
Cc: "yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: fetch from perforce
Date: Mon, 28 Oct 2013 09:35:50 +0000	[thread overview]
Message-ID: <17076206.HsEGNRlfMJ@helios> (raw)
In-Reply-To: <CAM1ZcoW7b8t-TFtUyZb=-PWDJNTPxbJuKA4CKBqc1izh+P7GUw@mail.gmail.com>

Hi Katu,

On Thursday 24 October 2013 10:09:50 Katu Txakur wrote:
> it took me longer than I thought because I couldn't find any documentation
> or comments in the code, but I finally got this working. The sintax to call
> it was, (there are actually 2 options, I took this one because I don't need
> user/passwd):
> 
> 
> P4PORT = "yourhost:yourportnumber"

Is this not able to be specified in the SRC_URI?

> FETCHCOMMAND_p4 = "p4"

We should really get a default value for FETCHCOMMAND_p4 defined in OE-Core 
(and perhaps also the default bitbake.conf bundled with bitbake itself).
 
> SRC_URI = " \
> p4://depot/myproject...;module=destinationfolderInWORKDIR/destinationsubfold
> er;changeslist=9952 \
> "
> 
> I used the perforce.py in the latest yocto distribution, 1.5-dora. The
> subfolders where being renamed wrongly, deleting the first 3 characters. I
> solved it using this patch:
> 
> diff --git poky/bitbake/lib/bb/fetch2/perforce.py_dora
> poky/bitbake/lib/bb/fetch2/perforce.py
> index fc4074d..d2ecccc 100644
> --- poky/bitbake/lib/bb/fetch2/perforce.py_dora
> +++ poky/bitbake/lib/bb/fetch2/perforce.py
> @@ -112,7 +112,7 @@ class Perforce(FetchMethod):
>          base = path
>          which = path.find('/...')
>          if which != -1:
> -            base = path[:which]
> +            base = path[:which-1]
> 
>          base = self._strip_leading_slashes(base)
> 
> I hope this helps to anyone trying to use this.

I see you've opened up a bug for this, thanks.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


  reply	other threads:[~2013-10-28  9:35 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-15 19:52 fetch from perforce Katu Txakur
2013-10-15 22:34 ` Paul Eggleton
2013-10-24  9:09   ` Katu Txakur
2013-10-28  9:35     ` Paul Eggleton [this message]
2013-10-28  9:38       ` Robert P. J. Day
2013-10-28  9:48         ` Paul Eggleton

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=17076206.HsEGNRlfMJ@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=katutxakurra@gmail.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.