Issue Details (XML | Word | Printable)

Key: APF-1117
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Unassigned
Reporter: Romke
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
AppFuse

appfuse:full-source removes .svn/tmp folder causing svn client to fail

Created: 09/Dec/08 01:37 PM   Updated: 09/Dec/08 02:58 PM   Resolved: 09/Dec/08 02:58 PM
Component/s: Tools - AMP
Affects Version/s: 2.0.2
Fix Version/s: 2.1.0-M1

File Attachments: 1. Text File RenamePackages.patch (1 kB)

Environment: MS Vista, possibly others


 Description  « Hide
appfuse:full-source goal removes empty folders, such as .svn/tmp. Some (many?) svn clients fail when the tmp folder is missing.

Scenario:
- check project into svn
- run appfuse:full-source
- notice that empty subfolders under .svn/ are missing

Cause:
A folder outside the old package is only created in the work folder if it contains one or more files, so the work folder will not contain empty folders.

Fix:
Retain empty folders, see attachment for suggestion

Side effect:
If the new package name is not 'org.*', then a number of empty org packages will be created



Matt Raible added a comment - 09/Dec/08 02:31 PM
Does this patch fix the issue entirely? If so, I'll commit it for 2.1.

Romke added a comment - 09/Dec/08 02:46 PM
It fixes the problem of the lost empty dirs, but their may be side effects that I haven't thought of.

An alternative approach would be to treat .svn folders differently from the rest, but I think that's less elegant.

3179 by  Matt Raible (1 file)
09/Dec/08 02:54 PM (20 months, 12 days ago)
APF-1117: Fixed full-source so it doesn't remove .svn/tmp. Thanks to Romke for the patch.
appfuse: trunk/plugins/appfuse-maven-plugin/src/main/java/org/appfuse/tool/RenamePackages.java 3179 history download (+11 -8) diffs

Matt Raible made changes - 09/Dec/08 02:58 PM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 2.1 [ 10031 ]
Resolution Fixed [ 1 ]