Re: .: officecfg/registry sd/source

On 18/09/12 21:01, Michael Stahl wrote:
> hi Stephan,
>
> On 18/09/12 20:46, Libreoffice Gerrit user wrote:
> >   officecfg/registry/cppheader.xsl      |   18 +++++++++---------
> >   officecfg/registry/files.mk           |    3 ++-
> >   sd/source/ui/remotecontrol/Server.cxx |    6 ++++--
> >   3 files changed, 15 insertions(+), 12 deletions(-)
> >
> > New commits:
> > commit ee5403b98bf0e89693a14f7a1ae5658d9e5eb730
> > Author: Michael Stahl <mstahl@redhat.com>
> > Date:   Tue Sep 18 20:41:56 2012 +0200
> >
> >      officecfg: generate header for Impress-sdremote, handle "-" in xcs name
> this was an urgent change to fix a non-building master, to handle the
> newly introduced "Office/Impress-sdremote.xcs"; but i wonder, is it
> perhaps intentional that schema files aren't allowed to have "-" in
> their names, considering that there are a bunch of .xcu files with names
> like Foo-bar.xcu where a Foo.xcs exists; should the
> Office/Impress-sdremote.xcs simply be renamed instead, and the xslt
> change reverted?
I should mention that the new config file fails to work 
(NoSuchElementException trying to access the config): while trying to 
fix it I renamed to ImpressSdRemote.xcs (which didn't fix things, but 
fits in better with the general naming scheme -- it seems that adding 
the file to postprocess/packregistry/makefile.mk was the key step).

I've commited the renaming, meaning the xslt changes can be reverted if 
needed.