[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH v2 1/1] package.bbclass: Remove unused variables and imports
From:       "Burton, Ross" <ross.burton () intel ! com>
Date:       2016-10-31 17:04:52
Message-ID: CAJTo0LZ6tiZGsTZ1WSRPa2wyY4oRY+5wLkisLN4cd6f1wmor1w () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On 26 October 2016 at 18:39, Ulf Magnusson <ulfalizer@gmail.com> wrote:

>              # Skip it if the target doesn't exist
>              try:
> -                s = os.stat(fpath)
> +                os.stat(fpath)
>              except OSError as e:
>                  (err, strerror) = e.args
>                  if err != errno.ENOENT:
>

Urgh, why not just use os.path.lexists()...   Can you change the code to do
that?

Ross

[Attachment #5 (text/html)]

<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 26 October \
2016 at 18:39, Ulf Magnusson <span dir="ltr">&lt;<a href="mailto:ulfalizer@gmail.com" \
target="_blank">ulfalizer@gmail.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div id=":15c" class="a3s aXjCH m158022896582464f">           \
# Skip it if the target doesn&#39;t exist<br>  try:<br>
-                        s = os.stat(fpath)<br>
+                        os.stat(fpath)<br>
                    except OSError as e:<br>
                          (err, strerror) = e.args<br>
                          if err != errno.ENOENT:</div></blockquote></div><br>Urgh, \
why not just use os.path.lexists()...    Can you change the code to do \
that?</div><div class="gmail_extra"><br></div><div \
class="gmail_extra">Ross</div></div>



-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic