Author: bodewig Date: Tue Aug 18 15:44:32 2009 New Revision: 805463 URL: http://svn.apache.org/viewvc?rev=805463&view=rev Log: the update attribute of the modified selector was ignored. PR 32597
Added: ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml (with props) Modified: ant/core/trunk/WHATSNEW ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java Modified: ant/core/trunk/WHATSNEW URL: http://svn.apache.org/viewvc/ant/core/trunk/WHATSNEW?rev=805463&r1=805462&r2=805463&view=diff ============================================================================== --- ant/core/trunk/WHATSNEW (original) +++ ant/core/trunk/WHATSNEW Tue Aug 18 15:44:32 2009 @@ -436,6 +436,9 @@ separator other than "/". Bugzilla Report 35649. + * The update attribute of the modified selector was ignored. + Bugzilla Report 32597. + Other changes: -------------- * The get task now also follows redirects from http to https Modified: ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java?rev=805463&r1=805462&r2=805463&view=diff ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java Tue Aug 18 15:44:32 2009 @@ -298,8 +298,6 @@ Cache defaultCache = new PropertiesfileCache(cachefile); Algorithm defaultAlgorithm = new DigestAlgorithm(); Comparator defaultComparator = new EqualComparator(); - update = true; - selectDirectories = true; // // ----- Set the main attributes, pattern '*' ----- Added: ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml?rev=805463&view=auto ============================================================================== --- ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml (added) +++ ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml Tue Aug 18 15:44:32 2009 @@ -0,0 +1,76 @@ +<?xml version="1.0"?> +<!-- + Licensed to the Apache Software Foundation (ASF) under one or more + contributor license agreements. See the NOTICE file distributed with + this work for additional information regarding copyright ownership. + The ASF licenses this file to You under the Apache License, Version 2.0 + (the "License"); you may not use this file except in compliance with + the License. You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. +--> +<project xmlns:au="antlib:org.apache.ant.antunit" default="antunit"> + + <import file="../../antunit-base.xml" /> + + <target name="tearDown" depends="antunit-base.tearDown"> + <delete file="cache.properties" quiet="true"/> + </target> + + <target name="testUpdate" + description="https://issues.apache.org/bugzilla/show_bug.cgi?id=32597"> + <mkdir dir="${input}"/> + <touch file="${input}/A"/> + <mkdir dir="${output}"/> + <pathconvert> + <fileset dir="${input}"> + <modified/> + </fileset> + </pathconvert> + <au:assertFileExists file="cache.properties"/> + <copy todir="${output}" file="cache.properties"/> + <echo file="${input}/A">Hello</echo> + <pathconvert> + <fileset dir="${input}"> + <modified update="false"/> + </fileset> + </pathconvert> + <au:assertFilesMatch + expected="${output}/cache.properties" + actual="cache.properties"/> + <pathconvert> + <fileset dir="${input}"> + <modified> + <param name="update" value="false"/> + </modified> + </fileset> + </pathconvert> + <au:assertFilesMatch + expected="${output}/cache.properties" + actual="cache.properties"/> + <pathconvert> + <fileset dir="${input}"> + <modified update="true"/> + </fileset> + </pathconvert> + <au:assertFilesDiffer + expected="${output}/cache.properties" + actual="cache.properties"/> + <copy todir="${output}" file="cache.properties"/> + <echo file="${input}/A">world</echo> + <pathconvert> + <fileset dir="${input}"> + <modified update="true"/> + </fileset> + </pathconvert> + <au:assertFilesDiffer + expected="${output}/cache.properties" + actual="cache.properties"/> + </target> +</project> Propchange: ant/core/trunk/src/tests/antunit/types/selectors/modified-test.xml ------------------------------------------------------------------------------ svn:eol-style = native