Hi Paul,

> -----Original Message-----
> From: [email protected]
> <[email protected]> On Behalf Of
> Manjukumar Harthikote Matha
> Sent: Monday, March 4, 2019 3:27 PM
> To: Paul Eggleton <[email protected]>; openembedded-
> [email protected]; Chandana Kalluri <[email protected]>
> Subject: Re: [OE-core] [OE-Core][master][PATCH] standard.py: Provide an
> additional option for devtool reset
> 
> Hi Paul,
> 
> > -----Original Message-----
> > From: [email protected]
> > [mailto:[email protected]] On Behalf Of
> > Paul Eggleton
> > Sent: Monday, March 04, 2019 12:17 PM
> > To: [email protected]; Chandana Kalluri
> > <[email protected]>
> > Subject: Re: [OE-core] [OE-Core][master][PATCH] standard.py: Provide
> > an additional option for devtool reset
> >
> > On Friday, 8 February 2019 2:57:32 PM NZDT Sai Hari Chandana Kalluri wrote:
> > > The devtool reset command cleans the sysroot for a recipe in workspace.
> > > It also removes the append file but leaves the source code as in
> > > workspace. The source is not cleaned intentionally and the user has
> > > to manually remove it before calling devtool modify again.
> > >
> > > Provide the user with an option to remove the source code from
> > > workspace by adding a flag to the devtool reset command. The
> > > --rm-source option for the devtool reset command will also clean the
> > > source code from the workspace along with the sysroot and the append file.
> > > Ex: devtool reset --rm-source zip or devtool reset -r zip
> >
> > When I wrote this I made a conscious decision not to delete the
> > source, in case the user has unsaved work in the source tree; at least
> > then the user has to take an explicit step (an additional rm -rf) in
> > order to delete it. You might argue that adding an option is the same
> > as that explicit step, but I am still hesitant as you might for
> > example pick a previously used command from your command history with
> > -r in it and only realise after you've run it that you've just blown away a 
> > bunch
> of your work.
> >
> > However, I have received the feedback from a number of people that
> > they find it annoying that they have to delete the source themselves
> > before being able to run devtool on the recipe again. I'd like to hear
> > from the wider community on the following two questions:
> >
> > 1) Does the convenience of having this kind of option outweigh the
> > potential danger of deleting unsaved work?
> >
> > 2) Are there issues that anyone is experiencing that force you to run
> > devtool clean/finish and then re-run devtool modify (or add, or
> > upgrade), making this kind of situation come up more frequently? Or is
> > it mostly that the source directories end up being left around to stumble 
> > over
> some time later?
> 
> We have seen source directories end up being left around (taking more disk
> space on certain projects), hence a solution to remove it might be optimal. 
> If the
> preference is not have "-r" option to avoid command history mistake, then a 
> full
> command like devtool reset --rm-source is acceptable as well
> 


Any update on this?  Is there a reason not to merge this feature?

Thanks,
Manju
-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to