* [PATCH] toaster: full path in variable history
@ 2014-03-18 12:32 Marius Avram
2014-03-18 14:45 ` Damian, Alexandru
0 siblings, 1 reply; 9+ messages in thread
From: Marius Avram @ 2014-03-18 12:32 UTC (permalink / raw)
To: toaster
When an evaluation was made for a configuration file the path to the
file was saved as a relative one. The change in this commit will save the
location as an absolute path. This way the user will have full information
regarding the location of the file where a variable was changed and the
line withing the file.
[YOCTO #5562]
Signed-off-by: Marius Avram <marius.avram@intel.com>
---
bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
index f4fb2aa..978ebe4 100644
--- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
+++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
@@ -144,7 +144,7 @@ def handle(fn, data, include):
# skip comments
if s[0] == '#':
continue
- feeder(lineno, s, fn, statements)
+ feeder(lineno, s, abs_fn, statements)
# DONE WITH PARSING... time to evaluate
data.setVar('FILE', abs_fn)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-18 12:32 [PATCH] toaster: full path in variable history Marius Avram
@ 2014-03-18 14:45 ` Damian, Alexandru
2014-03-18 15:15 ` Avram, Marius
0 siblings, 1 reply; 9+ messages in thread
From: Damian, Alexandru @ 2014-03-18 14:45 UTC (permalink / raw)
To: Marius Avram; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 1606 bytes --]
This patch is not in a toaster-only file, I think it affects other parts of
Bitbake code.
Can you please elaborate on the test plan you used to validate this change ?
Thank you,
Alex
On Tue, Mar 18, 2014 at 12:32 PM, Marius Avram <marius.avram@intel.com>wrote:
> When an evaluation was made for a configuration file the path to the
> file was saved as a relative one. The change in this commit will save the
> location as an absolute path. This way the user will have full information
> regarding the location of the file where a variable was changed and the
> line withing the file.
>
> [YOCTO #5562]
>
> Signed-off-by: Marius Avram <marius.avram@intel.com>
> ---
> bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> index f4fb2aa..978ebe4 100644
> --- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> +++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> @@ -144,7 +144,7 @@ def handle(fn, data, include):
> # skip comments
> if s[0] == '#':
> continue
> - feeder(lineno, s, fn, statements)
> + feeder(lineno, s, abs_fn, statements)
>
> # DONE WITH PARSING... time to evaluate
> data.setVar('FILE', abs_fn)
> --
> 1.7.9.5
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/toaster
>
--
Alex Damian
Yocto Project
SSG / OTC
[-- Attachment #2: Type: text/html, Size: 2762 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-18 14:45 ` Damian, Alexandru
@ 2014-03-18 15:15 ` Avram, Marius
2014-03-18 15:40 ` Damian, Alexandru
0 siblings, 1 reply; 9+ messages in thread
From: Avram, Marius @ 2014-03-18 15:15 UTC (permalink / raw)
To: Damian, Alexandru; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 2279 bytes --]
I only tested if the file_name fields from VariableHistory model is filled correctly by accessing:
http://localhost:8000/api/1.0/variablehistory which returns a JSON of the table.
I thought that replacing a relative path with an absolute path will not have any impact on the build system, especially that this part is used for variable history and logging.
Would you want to make this change specific only to toaster ?
From: Damian, Alexandru [mailto:alexandru.damian@intel.com]
Sent: Tuesday, March 18, 2014 4:45 PM
To: Avram, Marius
Cc: toaster@yoctoproject.org
Subject: Re: [Toaster] [PATCH] toaster: full path in variable history
This patch is not in a toaster-only file, I think it affects other parts of Bitbake code.
Can you please elaborate on the test plan you used to validate this change ?
Thank you,
Alex
On Tue, Mar 18, 2014 at 12:32 PM, Marius Avram <marius.avram@intel.com<mailto:marius.avram@intel.com>> wrote:
When an evaluation was made for a configuration file the path to the
file was saved as a relative one. The change in this commit will save the
location as an absolute path. This way the user will have full information
regarding the location of the file where a variable was changed and the
line withing the file.
[YOCTO #5562]
Signed-off-by: Marius Avram <marius.avram@intel.com<mailto:marius.avram@intel.com>>
---
bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
index f4fb2aa..978ebe4 100644
--- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
+++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
@@ -144,7 +144,7 @@ def handle(fn, data, include):
# skip comments
if s[0] == '#':
continue
- feeder(lineno, s, fn, statements)
+ feeder(lineno, s, abs_fn, statements)
# DONE WITH PARSING... time to evaluate
data.setVar('FILE', abs_fn)
--
1.7.9.5
--
_______________________________________________
toaster mailing list
toaster@yoctoproject.org<mailto:toaster@yoctoproject.org>
https://lists.yoctoproject.org/listinfo/toaster
--
Alex Damian
Yocto Project
SSG / OTC
[-- Attachment #2: Type: text/html, Size: 7247 bytes --]
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-18 15:15 ` Avram, Marius
@ 2014-03-18 15:40 ` Damian, Alexandru
2014-03-20 9:04 ` Marius Avram
0 siblings, 1 reply; 9+ messages in thread
From: Damian, Alexandru @ 2014-03-18 15:40 UTC (permalink / raw)
To: Avram, Marius; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 2780 bytes --]
Please validate that the reading of the config files is not affected by
deleting the recipe cache and trying to build from scratch.
This change cascades into the ast parser, not just logging and variable
history.
Alex
On Tue, Mar 18, 2014 at 3:15 PM, Avram, Marius <marius.avram@intel.com>wrote:
> I only tested if the file_name fields from VariableHistory model is filled
> correctly by accessing:
> http://localhost:8000/api/1.0/variablehistory which returns a JSON of
> the table.
>
> I thought that replacing a relative path with an absolute path will not
> have any impact on the build system, especially that this part is used for
> variable history and logging.
>
> Would you want to make this change specific only to toaster ?
>
> From: Damian, Alexandru [mailto:alexandru.damian@intel.com]
> Sent: Tuesday, March 18, 2014 4:45 PM
> To: Avram, Marius
> Cc: toaster@yoctoproject.org
> Subject: Re: [Toaster] [PATCH] toaster: full path in variable history
>
> This patch is not in a toaster-only file, I think it affects other parts
> of Bitbake code.
>
> Can you please elaborate on the test plan you used to validate this change
> ?
>
> Thank you,
> Alex
>
> On Tue, Mar 18, 2014 at 12:32 PM, Marius Avram <marius.avram@intel.com
> <mailto:marius.avram@intel.com>> wrote:
> When an evaluation was made for a configuration file the path to the
> file was saved as a relative one. The change in this commit will save the
> location as an absolute path. This way the user will have full information
> regarding the location of the file where a variable was changed and the
> line withing the file.
>
> [YOCTO #5562]
>
> Signed-off-by: Marius Avram <marius.avram@intel.com<mailto:
> marius.avram@intel.com>>
> ---
> bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> index f4fb2aa..978ebe4 100644
> --- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> +++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> @@ -144,7 +144,7 @@ def handle(fn, data, include):
> # skip comments
> if s[0] == '#':
> continue
> - feeder(lineno, s, fn, statements)
> + feeder(lineno, s, abs_fn, statements)
>
> # DONE WITH PARSING... time to evaluate
> data.setVar('FILE', abs_fn)
> --
> 1.7.9.5
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org<mailto:toaster@yoctoproject.org>
> https://lists.yoctoproject.org/listinfo/toaster
>
>
>
> --
> Alex Damian
> Yocto Project
> SSG / OTC
>
--
Alex Damian
Yocto Project
SSG / OTC
[-- Attachment #2: Type: text/html, Size: 4257 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-18 15:40 ` Damian, Alexandru
@ 2014-03-20 9:04 ` Marius Avram
2014-03-20 15:55 ` Damian, Alexandru
0 siblings, 1 reply; 9+ messages in thread
From: Marius Avram @ 2014-03-20 9:04 UTC (permalink / raw)
To: Damian, Alexandru; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 3683 bytes --]
I've made a build for core-image-minimal on top of commit
master:3c70ad664685716d8d805aebf211daff4a062d1a with my patch applied
inside a new build directory and I can confirm that the building process
finished successfully.
On 03/18/2014 05:40 PM, Damian, Alexandru wrote:
> Please validate that the reading of the config files is not affected
> by deleting the recipe cache and trying to build from scratch.
> This change cascades into the ast parser, not just logging and
> variable history.
>
> Alex
>
>
> On Tue, Mar 18, 2014 at 3:15 PM, Avram, Marius <marius.avram@intel.com
> <mailto:marius.avram@intel.com>> wrote:
>
> I only tested if the file_name fields from VariableHistory model
> is filled correctly by accessing:
> http://localhost:8000/api/1.0/variablehistory which returns a
> JSON of the table.
>
> I thought that replacing a relative path with an absolute path
> will not have any impact on the build system, especially that this
> part is used for variable history and logging.
>
> Would you want to make this change specific only to toaster ?
>
> From: Damian, Alexandru [mailto:alexandru.damian@intel.com
> <mailto:alexandru.damian@intel.com>]
> Sent: Tuesday, March 18, 2014 4:45 PM
> To: Avram, Marius
> Cc: toaster@yoctoproject.org <mailto:toaster@yoctoproject.org>
> Subject: Re: [Toaster] [PATCH] toaster: full path in variable history
>
> This patch is not in a toaster-only file, I think it affects other
> parts of Bitbake code.
>
> Can you please elaborate on the test plan you used to validate
> this change ?
>
> Thank you,
> Alex
>
> On Tue, Mar 18, 2014 at 12:32 PM, Marius Avram
> <marius.avram@intel.com
> <mailto:marius.avram@intel.com><mailto:marius.avram@intel.com
> <mailto:marius.avram@intel.com>>> wrote:
> When an evaluation was made for a configuration file the path to the
> file was saved as a relative one. The change in this commit will
> save the
> location as an absolute path. This way the user will have full
> information
> regarding the location of the file where a variable was changed
> and the
> line withing the file.
>
> [YOCTO #5562]
>
> Signed-off-by: Marius Avram <marius.avram@intel.com
> <mailto:marius.avram@intel.com><mailto:marius.avram@intel.com
> <mailto:marius.avram@intel.com>>>
> ---
> bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> index f4fb2aa..978ebe4 100644
> --- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> +++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
> @@ -144,7 +144,7 @@ def handle(fn, data, include):
> # skip comments
> if s[0] == '#':
> continue
> - feeder(lineno, s, fn, statements)
> + feeder(lineno, s, abs_fn, statements)
>
> # DONE WITH PARSING... time to evaluate
> data.setVar('FILE', abs_fn)
> --
> 1.7.9.5
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org
> <mailto:toaster@yoctoproject.org><mailto:toaster@yoctoproject.org
> <mailto:toaster@yoctoproject.org>>
> https://lists.yoctoproject.org/listinfo/toaster
>
>
>
> --
> Alex Damian
> Yocto Project
> SSG / OTC
>
>
>
>
> --
> Alex Damian
> Yocto Project
> SSG / OTC
[-- Attachment #2: Type: text/html, Size: 6972 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-20 9:04 ` Marius Avram
@ 2014-03-20 15:55 ` Damian, Alexandru
0 siblings, 0 replies; 9+ messages in thread
From: Damian, Alexandru @ 2014-03-20 15:55 UTC (permalink / raw)
To: Marius Avram; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 3345 bytes --]
Taken in.
Alex
On Thu, Mar 20, 2014 at 9:04 AM, Marius Avram <marius.avram@intel.com>wrote:
> I've made a build for core-image-minimal on top of commit
> master:3c70ad664685716d8d805aebf211daff4a062d1a with my patch applied
> inside a new build directory and I can confirm that the building process
> finished successfully.
>
>
> On 03/18/2014 05:40 PM, Damian, Alexandru wrote:
>
> Please validate that the reading of the config files is not affected by
> deleting the recipe cache and trying to build from scratch.
> This change cascades into the ast parser, not just logging and variable
> history.
>
> Alex
>
>
> On Tue, Mar 18, 2014 at 3:15 PM, Avram, Marius <marius.avram@intel.com>wrote:
>
>> I only tested if the file_name fields from VariableHistory model is
>> filled correctly by accessing:
>> http://localhost:8000/api/1.0/variablehistory which returns a JSON of
>> the table.
>>
>> I thought that replacing a relative path with an absolute path will not
>> have any impact on the build system, especially that this part is used for
>> variable history and logging.
>>
>> Would you want to make this change specific only to toaster ?
>>
>> From: Damian, Alexandru [mailto:alexandru.damian@intel.com]
>> Sent: Tuesday, March 18, 2014 4:45 PM
>> To: Avram, Marius
>> Cc: toaster@yoctoproject.org
>> Subject: Re: [Toaster] [PATCH] toaster: full path in variable history
>>
>> This patch is not in a toaster-only file, I think it affects other parts
>> of Bitbake code.
>>
>> Can you please elaborate on the test plan you used to validate this
>> change ?
>>
>> Thank you,
>> Alex
>>
>> On Tue, Mar 18, 2014 at 12:32 PM, Marius Avram <marius.avram@intel.com
>> <mailto:marius.avram@intel.com>> wrote:
>> When an evaluation was made for a configuration file the path to the
>> file was saved as a relative one. The change in this commit will save the
>> location as an absolute path. This way the user will have full information
>> regarding the location of the file where a variable was changed and the
>> line withing the file.
>>
>> [YOCTO #5562]
>>
>> Signed-off-by: Marius Avram <marius.avram@intel.com<mailto:
>> marius.avram@intel.com>>
>> ---
>> bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
>> b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
>> index f4fb2aa..978ebe4 100644
>> --- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
>> +++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
>> @@ -144,7 +144,7 @@ def handle(fn, data, include):
>> # skip comments
>> if s[0] == '#':
>> continue
>> - feeder(lineno, s, fn, statements)
>> + feeder(lineno, s, abs_fn, statements)
>>
>> # DONE WITH PARSING... time to evaluate
>> data.setVar('FILE', abs_fn)
>> --
>> 1.7.9.5
>>
>> --
>> _______________________________________________
>> toaster mailing list
>> toaster@yoctoproject.org<mailto:toaster@yoctoproject.org>
>> https://lists.yoctoproject.org/listinfo/toaster
>>
>>
>>
>> --
>> Alex Damian
>> Yocto Project
>> SSG / OTC
>>
>
>
>
> --
> Alex Damian
> Yocto Project
> SSG / OTC
>
>
>
--
Alex Damian
Yocto Project
SSG / OTC
[-- Attachment #2: Type: text/html, Size: 7054 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] toaster: full path in variable history
@ 2014-03-11 16:37 Marius Avram
2014-03-21 13:52 ` Richard Purdie
0 siblings, 1 reply; 9+ messages in thread
From: Marius Avram @ 2014-03-11 16:37 UTC (permalink / raw)
To: toaster
When an evaluation was made for a configuration file the path to the
file was saved as a relative one. The change in this commit will save the
location as an absolute path. This way the user will have full information
regarding the location of the file where a variable was changed and the
line withing the file.
[YOCTO #5562]
Signed-off-by: Marius Avram <marius.avram@intel.com>
---
bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bitbake/lib/bb/parse/parse_py/ConfHandler.py b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
index f4fb2aa..978ebe4 100644
--- a/bitbake/lib/bb/parse/parse_py/ConfHandler.py
+++ b/bitbake/lib/bb/parse/parse_py/ConfHandler.py
@@ -144,7 +144,7 @@ def handle(fn, data, include):
# skip comments
if s[0] == '#':
continue
- feeder(lineno, s, fn, statements)
+ feeder(lineno, s, abs_fn, statements)
# DONE WITH PARSING... time to evaluate
data.setVar('FILE', abs_fn)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH] toaster: full path in variable history
2014-03-11 16:37 Marius Avram
@ 2014-03-21 13:52 ` Richard Purdie
2014-03-24 15:29 ` Damian, Alexandru
0 siblings, 1 reply; 9+ messages in thread
From: Richard Purdie @ 2014-03-21 13:52 UTC (permalink / raw)
To: Marius Avram; +Cc: toaster
On Tue, 2014-03-11 at 18:37 +0200, Marius Avram wrote:
> When an evaluation was made for a configuration file the path to the
> file was saved as a relative one. The change in this commit will save the
> location as an absolute path. This way the user will have full information
> regarding the location of the file where a variable was changed and the
> line withing the file.
>
> [YOCTO #5562]
>
> Signed-off-by: Marius Avram <marius.avram@intel.com>
> ---
> bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
I've taken this since I think it should be reasonable however I did have
to change the commit message since this doesn't touch toaster, it
touched ConfHandler.
Cheers,
Richard
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] toaster: full path in variable history
2014-03-21 13:52 ` Richard Purdie
@ 2014-03-24 15:29 ` Damian, Alexandru
0 siblings, 0 replies; 9+ messages in thread
From: Damian, Alexandru @ 2014-03-24 15:29 UTC (permalink / raw)
To: Richard Purdie; +Cc: toaster@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 1246 bytes --]
Thank you for taking care of this.
I've got this through some testing and I could not find fault :)
Thanks,
Alex
On Fri, Mar 21, 2014 at 1:52 PM, Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2014-03-11 at 18:37 +0200, Marius Avram wrote:
> > When an evaluation was made for a configuration file the path to the
> > file was saved as a relative one. The change in this commit will save the
> > location as an absolute path. This way the user will have full
> information
> > regarding the location of the file where a variable was changed and the
> > line withing the file.
> >
> > [YOCTO #5562]
> >
> > Signed-off-by: Marius Avram <marius.avram@intel.com>
> > ---
> > bitbake/lib/bb/parse/parse_py/ConfHandler.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> I've taken this since I think it should be reasonable however I did have
> to change the commit message since this doesn't touch toaster, it
> touched ConfHandler.
>
> Cheers,
>
> Richard
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/toaster
>
--
Alex Damian
Yocto Project
SSG / OTC
[-- Attachment #2: Type: text/html, Size: 2051 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-03-24 15:29 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-18 12:32 [PATCH] toaster: full path in variable history Marius Avram
2014-03-18 14:45 ` Damian, Alexandru
2014-03-18 15:15 ` Avram, Marius
2014-03-18 15:40 ` Damian, Alexandru
2014-03-20 9:04 ` Marius Avram
2014-03-20 15:55 ` Damian, Alexandru
-- strict thread matches above, loose matches on Subject: below --
2014-03-11 16:37 Marius Avram
2014-03-21 13:52 ` Richard Purdie
2014-03-24 15:29 ` Damian, Alexandru
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.