Andreas Hartmann wrote:
> Jörn Nettingsmeier schrieb:
>> hi *!
>>
>> i'm trying to update the Instantiator class. i have
it working, but
>> there are a number of hard-coded strings in there
i'd like to get rid
>> of. so i'd like to do:
>
>
> Hi Jörn,
>
> you're hitting a fundamental problem which I always
wanted to tackle,
> but IMO can be deferred until 1.4 is out.
>
> In 1.2, publications were a static concept. You
configure your
> publications in the file system and start the server.
>
> In 1.4, publication templating and instantiation
introduced the
> dynamic addition of publications. But the publication
concept was
> never updated to be dynamic, so we have something in
between now.
>
> IMO we should (someday) extend the API so that
publications can be
> created and configured using Java code and not by
copying and
> manipulating the configuration files.
>
> The location and contents of the config file is an
implementation
> detail, that's why its not part of the API and you
can't access
> it in the Instantiator. Until we have extended the
publication
> interface, I'd say we leave the hard-coded strings as
they are.
just to clarify: i don't want to change anything
fundamental, i just
want to reduce duplication of information. for instance, if
details of
the publication.xconf are defined by string constants in
PublicationConfiguration, it makes sense to use those
constants in the
instantiator as well, and not introduce new constants that
can go out of
sync. those changes are pretty local and unintrusive.
i'm absolutely not going to implement any funky publication
abstractions
at this point in time, i only want to use constants that are
already in
place.
is this build problem i'm seeing part of the
"fundamental problem" you
describe? if so, can you give details? i'm totally clueless
when it
comes to java dependency management.
thanks,
jörn
------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribe lenya.apache.org
For additional commands, e-mail: dev-help lenya.apache.org
|