Bug 7741

Summary: Improve how we handle the no toasterconf.json files found situation
Product: [Build System, Metadata & Runtime] Toaster Reporter: Belen Barros Pena <belen.barros.pena>
Component: toasterAssignee: Michael Wood <michael.g.wood>
Status: VERIFIED FIXED QA Contact: Bogdan Alexandru Voiculescu <bogdanx.a.voiculescu>
Severity: enhancement    
Priority: Medium CC: alexandru.damian, belen.barros.pena, bluelightning, bogdanx.a.voiculescu, cristiana.voicu, jessica.zhang, michael.g.wood, randy.e.witt, srifenbark
Version: 1.8   
Target Milestone: 1.9 M1   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Belen Barros Pena 2015-05-12 12:50:09 UTC
If everything goes well during the setup process, the Toaster script will show you one or more configuration files you can choose from. If no configuration files are found, however, the only option we provide to users is to proceed without importing a configuration file, which renders Toaster pretty much useless.

We should better handle the case when no configuration files are found, by printing something like this: 

Toaster can use a single predefined configuration file to set up default project settings and layer configuration sources. 

Toaster has not found any configuration files, but you can specify the path to a configuration file manually. Configuration files should be inside the directory where you cloned the source you are using to run Toaster. 

If you cloned the poky repository from http://git.yoctoproject.org/git/poky, the configuration file you should use is "meta-yocto/conf/toasterconf.json".

If you cloned the openembedded-core repository from git://git.openembedded.org/openembedded-core, the configuration file you should use is "meta/conf/toasterconf.json"

Type the full path to a configuration file, or 0 to proceed without importing any file: 

Once more, Randy and Scott, feel free to suggest improvements to the above.
Comment 1 Belen Barros Pena 2015-05-15 10:32:00 UTC
And since we are on it, let's exclude the toasterconf.json files inside any directory with name including 'toaster_cloned'.

That will exclude the files that come with layer checkouts, and should reduce the list dramatically if you are using Toaster locally.
Comment 2 Scott Rifenbark 2015-05-15 13:23:26 UTC
(In reply to comment #0)
> If everything goes well during the setup process, the Toaster script will
> show you one or more configuration files you can choose from. If no
> configuration files are found, however, the only option we provide to users
> is to proceed without importing a configuration file, which renders Toaster
> pretty much useless.
> 
> We should better handle the case when no configuration files are found, by
> printing something like this: 
> 
> Toaster can use a single predefined configuration file to set up default
> project settings and layer configuration sources. 
> 
> Toaster has not found any configuration files, but you can specify the path
> to a configuration file manually. Configuration files should be inside the
> directory where you cloned the source you are using to run Toaster. 
> 
> If you cloned the poky repository from http://git.yoctoproject.org/git/poky,
> the configuration file you should use is "meta-yocto/conf/toasterconf.json".
> 
> If you cloned the openembedded-core repository from
> git://git.openembedded.org/openembedded-core, the configuration file you
> should use is "meta/conf/toasterconf.json"

You should have a period at the end of the above sentence.

> 
> Type the full path to a configuration file, or 0 to proceed without
> importing any file: 

Maybe this change - "Type the full path to a configuration file, or type 0 to proceed without importing any file:"

> 
> Once more, Randy and Scott, feel free to suggest improvements to the above.
Comment 3 Michael Wood 2015-06-26 10:43:34 UTC
(In reply to comment #1)
> And since we are on it, let's exclude the toasterconf.json files inside any
> directory with name including 'toaster_cloned'.
> 
> That will exclude the files that come with layer checkouts, and should
> reduce the list dramatically if you are using Toaster locally.

Landed a fix for this.

 http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=62288a66896416beaef1d85aeefc8dab0835f4f8

If there are other items not fixed for this bug please could a new bug be opened.
Comment 4 Bogdan Alexandru Voiculescu 2015-09-07 11:45:33 UTC
verified