Re: [ptxdist] [PATCH] openocd: new package for openocd-0.8.0 (host, target)

[Michael Olbrich <m.olbrich@pengutronix.de>]

On Wed, Oct 01, 2014 at 04:25:21PM +0200, Andreas Pretzsch wrote:
> > > +#	@$(call install_tree, openocd, 0, 0, -, /usr/share/openocd/)
> > 
> > don't add comments like this either remove this or actually install any
> > useful files.
> 
> >From your other reply:
> > > +	@$(call install_copy, openocd, 0, 0, 0755, -, /usr/bin/openocd)
> > > > +#	@$(call install_tree, openocd, 0, 0, -, /usr/share/openocd/)
> > > 
> > > Intentionally disabled. 700kB generic extra data on the target makes no
> > > sense. I doubt there won't be one real use case where the makefile is
> > > not pulled into platform local rules and adapted to copy only the 5
> > > relevant files from this tree. So this is more of a reminder where to
> > > look, so I kept is as comment.
> > 
> > I missed this when I commented the patch. Are there any files in
> > /usr/share/openocd/ that are required on the target?
> 
> Not strictly out from there.
> 
> Beside device-specific firmware and examples, it contains some 500
> config files and tcl scripts. Core and SoC specific configs and setups,
> interface (debug adapter) specific configs, RAM and flash layout
> definitions, (eval) board init routines, some helpers, and so on. For
> the myriad targets OpenOCD supports.
> 
> In practice on the target, one will use maybe 5 out of them, sometimes
> even merged into one file, and also adapt it to the specific hardware.
> So a mixture of a few selected ones plus maybe some not even in the
> official package.
> 
> I see no sensible way for a preselection here, and in 99% of the cases,
> it will be project specific. So no pre-install. But I like to keep the
> comment for those who will adapt it to their needs, to ease things.
> 
> What I'll do in v2 is to switch from install_tree to
> install_alternative_tree, missed that one. But still commented out.

Use install_alternative_tree and add an option for it.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de