You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Brett Porter <br...@apache.org> on 2009/08/10 22:41:24 UTC
Re: svn commit: r802166 - /maven/resources/trunk/apache-source-release-assembly-descriptor/src/main/resources/assemblies/source-release.xml
On 07/08/2009, at 4:11 PM, jdcasey@apache.org wrote:
> + <!-- misc -->
> <exclude>**/cobertura.ser</exclude>
> + <exclude>**/*.bak</exclude>
> + <exclude>**/*.diff</exclude>
> + <exclude>**/*.patch</exclude>
> + <exclude>**/*.log</exclude>
I feel like these could start knocking out legitimate files.
Are they really necessary since it is usually built from a clean
checkout?
- Brett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r802166 - /maven/resources/trunk/apache-source-release-assembly-descriptor/src/main/resources/assemblies/source-release.xml
Posted by Brian Fox <br...@infinity.nu>.
The cobertura one yes because it would be produced potentially by the
tests. The others probably have less harm.
On Mon, Aug 10, 2009 at 4:41 PM, Brett Porter<br...@apache.org> wrote:
>
> On 07/08/2009, at 4:11 PM, jdcasey@apache.org wrote:
>
>> + <!-- misc -->
>> <exclude>**/cobertura.ser</exclude>
>> + <exclude>**/*.bak</exclude>
>> + <exclude>**/*.diff</exclude>
>> + <exclude>**/*.patch</exclude>
>> + <exclude>**/*.log</exclude>
>
> I feel like these could start knocking out legitimate files.
>
> Are they really necessary since it is usually built from a clean checkout?
>
> - Brett
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Source-release excludes patterns [was: Re: svn commit: r802166]
Posted by John Casey <jd...@commonjava.org>.
This is something I wanted to bring up on the dev list, actually. I
simply went through all of the source-release assembly descriptors used
to release various plugins, and compiled all of the exclusion patterns.
I'm not sure it's a good idea to leave these or any of them out, but I
can definitely see that at least in the case of the cobertura.ser file,
it is probably not the intention of the user to leave it in.
As for the rest, I'm fine with dropping them; I wanted to get the
opinion of others...
Brett Porter wrote:
>
> On 07/08/2009, at 4:11 PM, jdcasey@apache.org wrote:
>
>> + <!-- misc -->
>> <exclude>**/cobertura.ser</exclude>
>> + <exclude>**/*.bak</exclude>
>> + <exclude>**/*.diff</exclude>
>> + <exclude>**/*.patch</exclude>
>> + <exclude>**/*.log</exclude>
>
> I feel like these could start knocking out legitimate files.
>
> Are they really necessary since it is usually built from a clean checkout?
>
> - Brett
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org