Modify

Opened 16 years ago

Closed 16 years ago

#2585 closed defect (fixed)

[PATCH?] Changeset [1592] breaks JOSM compilation

Reported by: avarab@… Owned by: team
Priority: blocker Milestone:
Component: Core Version: latest
Keywords: Cc:

Description

ant clean dist is required when changing function definitions like pasteData in [1592].

Attachments (1)

DuplicateAction-pasteData.patch (575 bytes ) - added by avarab@… 16 years ago.
Just set the second parameter to null when we have no source layer?

Download all attachments as: .zip

Change History (4)

by avarab@…, 16 years ago

Just set the second parameter to null when we have no source layer?

comment:1 by stoecker, 16 years ago

Resolution: fixed
Status: newclosed

Set it to current layer instead.

In r1593.

I hate it that ant/java cannot track deps properly.

in reply to:  1 comment:2 by matt_gnu, 16 years ago

Resolution: fixed
Status: closedreopened

curretn SVN r1593 does'nt compile yet:

[javac] .../src/org/openstreetmap/josm/actions/DuplicateAction.java:29: package Main does not exist

Index: src/org/openstreetmap/josm/actions/DuplicateAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/DuplicateAction.java	(Revision 1593)
+++ src/org/openstreetmap/josm/actions/DuplicateAction.java	(Arbeitskopie)
@@ -8,6 +8,7 @@
 import java.awt.event.KeyEvent;
 import java.util.Collection;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.CopyAction;
 import org.openstreetmap.josm.actions.PasteAction;
 import org.openstreetmap.josm.data.SelectionChangedListener;

seems a proper fix, without really knowing the josm-code ;-)

comment:3 by stoecker, 16 years ago

Resolution: fixed
Status: reopenedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.