Opened 3 years ago
Closed 3 years ago
#5303 closed defect (worksforme)
JOSM removes all deleted nodes from local data set after upload, rather than only those that have been uploaded.
| Reported by: | Matthew Bell | Owned by: | Matthew Bell |
|---|---|---|---|
| Priority: | normal | Component: | Core |
| Version: | latest | Keywords: | Patch |
| Cc: |
Description
Attachments (4)
Change History (12)
Changed 3 years ago by Matthew Bell
Changed 3 years ago by Matthew Bell
Changed 3 years ago by Matthew Bell
Changed 3 years ago by Matthew Bell
comment:1 Changed 3 years ago by Matthew Bell
- Version set to latest
comment:2 Changed 3 years ago by stoecker
- Summary changed from JOSM removes all deleted nodes from local data set after upload, rather than only those that have been uploaded. to [PATCH] JOSM removes all deleted nodes from local data set after upload, rather than only those that have been uploaded.
comment:3 Changed 3 years ago by stoecker
comment:4 Changed 3 years ago by stoecker
- Owner changed from team to Matthew Bell
- Status changed from new to needinfo
- Summary changed from [PATCH] JOSM removes all deleted nodes from local data set after upload, rather than only those that have been uploaded. to JOSM removes all deleted nodes from local data set after upload, rather than only those that have been uploaded.
Only partially applied. Please fix the remaining patches or close ticket when already fixed.
comment:5 follow-up: ↓ 7 Changed 3 years ago by Matthew Bell
Oh come on, either apply them properly or not at all.
The patches in this ticket were based on r3416, which I believe was current when I posted them here. As noted, they were updated from the ones in the email. I've just checked them again and they apply cleanly. I have no idea why they don't apply for you; perhaps you need to apply them in the right order. I'm sorry, but I'm not going to clean up the mess you've made with that commit; it's really not that hard to merge patches if you've made conflicting changes in one or two commits.
comment:6 Changed 3 years ago by stoecker
Because inbetween SVn changed. You patches aren't the only ones. Try against newest SVN and you see they wont apply.
comment:7 in reply to: ↑ 5 Changed 3 years ago by bastiK
Replying to Matthew Bell:
Oh come on, either apply them properly or not at all.
The patches in this ticket were based on r3416, which I believe was current when I posted them here. As noted, they were updated from the ones in the email. I've just checked them again and they apply cleanly. I have no idea why they don't apply for you; perhaps you need to apply them in the right order. I'm sorry, but I'm not going to clean up the mess you've made with that commit; it's really not that hard to merge patches if you've made conflicting changes in one or two commits.
Please don't take it personal. It is always easier for the author of the patch to fix things than for the one who happens to commit it. And Stoecker hasn't made any mess that you could clean up. He simply applied 2 patches that where submitted before your's. It's not his job to fix subsequent contributions.
comment:8 Changed 3 years ago by stoecker
- Resolution set to worksforme
- Status changed from needinfo to closed



These are now based on latest SVN head.