← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH v3 5/5] [Build] No need to use the source tree for temporary files.

 

On Wed, Sep 21, 2016 at 09:04:03PM +0200, Niki Guldbrand wrote:
> On ons, 2016-09-21 at 20:13 +0200, jp charras wrote:
> > Le 21/09/2016 à 19:33, Niki Guldbrand a écrit :
> > > 
> > > On ons, 2016-09-21 at 12:32 +0200, jp charras wrote:
> > > > 
> > > > Le 21/09/2016 à 00:21, Niki Guldbrand a écrit :
> > > > > 
> > > > > 
> > > > > 
> > > > > * Move the generated PNG and temporary files to the build tree,
> > > > > but
> > > > >   keep the generated c files in the source tree for easier
> > > > >   addition/updates in tree.
> > > > > * Syncronize .gitignore to the changes.
> > > > > 
> > > > > Signed-off-by: Niki Guldbrand <niki.guldbrand@xxxxxxxxx>
> > > > > ---
> > > > >  .gitignore                 | 2 --
> > > > >  bitmaps_png/CMakeLists.txt | 5 ++---
> > > > >  2 files changed, 2 insertions(+), 5 deletions(-)
> > > > > 
> > > > 
> > > > I agree for temporary files.
> > > > 
> > > > I fully disagree to move generated PNG files to the build tree.
> > > > 
> > > > As a icon maintainer and a menu/toolbar maintainer, I am very
> > > > *happy*
> > > > to easily find them in source
> > > > tree, for instance when I want to create a new icon, or choose an
> > > > existing icon for a new menu item.
> > > > 
> > > > Build tree is subject to be deleted, not files in source tree.
> > > > 
> > > Well they are still available, you just have to look in the build
> > > tree
> > > instead, so in my view it's a none issue, as they will be recreated
> > > if
> > > needed.
> > > 
> > > And you still have the source files avalible (svg), to look at.
> > 
> > Are you joking?
> > There are near 500 icons.
> > 
> > Only png files can be displayed as images when exploring a folder,
> > not svg files.
> 
> Well I don't know what OS you normally use, but here (Linux - Arch) I
> see the svg files when browsing folders the same way as I would if they
> were png's or jpeg's.

I don't have too terribly much to say about this...but surely whether
your own OS and software previews them should be your own problem to
solve? It's not appropriate to litter the source tree used by all the
developers because it makes it easier for you personally to look through
previews...get a better directory viewer.

> > 
> > Moreover I wrote:
> > "Build tree is subject to be deleted, not files in source tree."
> > Clearly, you not read me.
> 
> Yes I did, but that doesn't mean that I have the same oppinion as you
> do, that is the nice thing about diversity in OSS, different ideas
> meet, and progress is made :-)
> 
> > > I was even considering removing the 'if( MAINTAIN_PNGS ) all
> > > together,
> > > and replace it with something that lookes if the cpp file was older
> > > than the svg, and in that case regenerate the cpp file and not keep
> > > the
> > > png's around.
> > 
> > Forget it.
> 
> Well you haven't seen my patch yet, and allready dismissing it, but see
> my answer above.
> 
> > > I'm also considering moving the svg's to the resources folder,
> > > because
> > > they are resources, and not source files, but that may just be my
> > > view.
> > > 
> > 
> > The current tree was designed by guys who know what they are doing,
> > not beginners.
> > It can be modified, but with good reasons.
> 
> I haven't said anything about the people that designed it, but still,
> there might be people out there with other ideas, might not be the
> best, but could be a middle ground for improvement.
> 
> I don't know if "not beginners" is ment as an insult or what, but, I
> don't know you, and you don't know me (and each others background for
> that matter), so my proposal is to keep the conversation civil, and not
> start calling each other names or something like that, and keep it
> technical and productive.
> 
> I know that I might have radical ideas in your and maybe others eyes,
> but I share them, and hope that we can find a common middle ground that
> we can agree on and that improves kicad.
> 
> > svg files are sources, like html or .fbp files
> 
> They are also image files, although they are not binary in nature, they
> are XML base, but they are still a resource ( in my view).
> 
> > Why some guys are thinking only cpp files are sources. This is false.
> > Sources files can be written in any language (svg *is* a language)
> 
> I don't agree with you here, as I see it svg's is a file format, and a
> subset of XML.
> 
> A language is in my view something where you have eg. for, if, while
> statements, and other constructs like that.
> 
> > Clearly you have never maintained icon files.
> 
> No I have not, you are correct on that point.
> 
> > Maintain is not just build file: it is most of time create, choose
> > and edit files.
> 
> I agree on that point too.
> 
> > Still, I am in the impression you did not really read my previous
> > mail.
> 
> As I wrote further up, yes I did, but our ideas and visions are
> proberbly not the same.
> 
> I'm not here to derail the project, I'm here to help the project along,
> and when new people come along there is a new pool of ideas that could
> be useful to the project.
> 
> Best regards
> 
> Niki
> 
> -- 
> Niki Guldbrand <niki.guldbrand@xxxxxxxxx>
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp


Follow ups

References