[jboss-as7-dev] [jboss-as] Move non-standard config files to the docs/examples/configs directory (#790)

[Jaikiran Pai]

I don't mind moving them to docs/examples/configs, but moving them 
there, is as good as having a untested config which can get outdated and 
might not be functional if someone tries to use it. But I don't think we 
even tested those extra configs even if those were in 
standalone/configuration or domain/configuration (did we test the 
standalone-xts.xml one for example?). So yeah, no objection moving those 
out.

-Jaikiran
On Saturday 03 December 2011 12:05 AM, Brian Stansberry wrote:
> FYI, the pull request below is moving config files other than the
> default one, the full profile one and the HA one to
> docs/examples/configs.  This was done on my recommendation (i.e. blame
> me) in order to reduce clutter in the configuration/ dir. Otherwise over
> time we are going to end up with a multitude of config file variants.
> (Carlo has a pull request out there with another one.)
>
> Speak now or forever hold your peace. ;)
>
> On 12/2/11 11:49 AM, Kabir Khan wrote:
>> It looks like ParseAndMarshalModelsTestCase never tested the standalone-hornetq-colocated.xml file. I fixed it up a bit but created https://issues.jboss.org/browse/AS7-2901
>>
>> You can merge this Pull Request by running:
>>
>>     git pull https://github.com/kabir/jboss-as special-configs-in-docs
>>
>> Or you can view, comment on it, or merge it online at:
>>
>>     https://github.com/jbossas/jboss-as/pull/790
>>
>> -- Commit Summary --
>>
>> * Move non-standard config files to the docs/examples/configs directory
>>
>> -- File Changes --
>>
>> A build/src/main/resources/docs/examples/configs/domain-osgi-only.xml (0)
>> M build/src/main/resources/docs/examples/configs/standalone-hornetq-colocated.xml (17)
>> A build/src/main/resources/docs/examples/configs/standalone-osgi-only.xml (0)
>> A build/src/main/resources/docs/examples/configs/standalone-xts.xml (0)
>> M testsuite/integration/smoke/src/test/java/org/jboss/as/test/smoke/embedded/parse/ParseAndMarshalModelsTestCase.java (39)
>>
>> -- Patch Links --
>>
>>     https://github.com/jbossas/jboss-as/pull/790.patch
>>     https://github.com/jbossas/jboss-as/pull/790.diff
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/jbossas/jboss-as/pull/790
>> _______________________________________________
>> jbossas-pull-requests mailing list
>> jbossas-pull-requests at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jbossas-pull-requests
>
>