Bug 923960 - Review Request: JMapViewer - JMapViewer is a java component to integrate an OSM map view into your Java application.
Summary: Review Request: JMapViewer - JMapViewer is a java component to integrate an O...
Status: CLOSED DUPLICATE of bug 1092629
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2013-03-20 19:34 UTC by hannes
Modified: 2014-05-01 09:59 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-02-03 07:40:07 UTC
Type: ---

Attachments (Terms of Use)

Description hannes 2013-03-20 19:34:02 UTC
Spec URL: http://hannes.fedorapeople.org/JMapViewer.spec
SRPM URL: http://hannes.fedorapeople.org/JMapViewer-2013.02.10-1.fc18.src.rpm
Description: JMapViewer is a Java Swing component for integrating OSM maps 
into your Java application. 
JMapViewer allows you to set markers on the map or zoom 
to a specific location on the map.
Fedora Account System Username: hannes

I need to remove the probably trademarked logo from the upstream release package, that's why I need to create a new tar ball without the logo and the relevant code.

Comment 2 Volker Fröhlich 2013-03-27 15:38:54 UTC
JMapViewer.spec:32: W: setup-not-quiet
JMapViewer.spec:52: W: mixed-use-of-spaces-and-tabs (spaces: line 52, tab: line 2)

The description commonly doesn't repeat the name and doesn't have a period at the end. The line is also too long, I guess.

How did you find out this software was GPLv2? The headers only say "GPL" and so does the website.

Buildroot is no longer necessary; same goes for defattr. Use the name macro in Source1.

Comment 3 hannes 2013-03-27 19:22:29 UTC
OK, fixed it. I think the license was a leftover from my spec template.
Spec URL: http://hannes.fedorapeople.org/JMapViewer.spec
SRPM URL: http://hannes.fedorapeople.org/JMapViewer-2013.03.22-2.fc18.src.rpm

Comment 4 gil cattaneo 2013-08-24 18:47:37 UTC
you should remove 
BuildRequires:	ant-nodeps
BuildRequires:	ant-trax
ant dont provides anymore these libraries
you should remove also

WARNING: Cannot download url: http://svn.openstreetmap.org/applications/viewer/JMapViewer/releases/2013-03-22/JMapViewer-nopatents.tar.gz
need only JMapViewer-nopatents.tar.gz
find -name '*.class' -exec rm -f '{}' \;
find -name '*.jar' -exec rm -f '{}' \;
find -name '*.zip' -exec rm -f '{}' \;
find -name '*.class' -delete
find -name '*.jar' -delete
find -name '*.zip' -delete
or edit JMapViewer_create_tarball_and_remove_bing.sh for remove bunlded libraries
remove all mixed-use-of-spaces-and-tabs
can you provides also javadoc package?

Comment 5 gil cattaneo 2013-08-24 19:02:40 UTC
please, use %setup -q -c -n %{name}

Comment 6 gil cattaneo 2013-08-24 19:05:56 UTC
summary should be: "Java Open Street Map Viewer"

Comment 7 gil cattaneo 2013-09-22 21:41:03 UTC

Comment 8 Volker Fröhlich 2014-01-31 21:20:21 UTC
Last response from hannes is from March last year. Hannes, are you still interested?

Comment 9 hannes 2014-02-03 05:48:22 UTC
Well, I am not really interested any more. I released ownership of freemind, so there's no need to get the deps in.
I just don't have enough time to maintain java packages.
So if anyone wants to take it, feel free.

Comment 10 Volker Fröhlich 2014-05-01 09:59:33 UTC

*** This bug has been marked as a duplicate of bug 1092629 ***

Note You need to log in before you can comment on or make changes to this bug.