[Bug 640205] Review Request: visualvm - Lightweight profiler that integrates many command-line JDK tools

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=640205

--- Comment #14 from jiri vanek <jvanek@xxxxxxxxxx> 2010-10-08 09:49:55 EDT ---
(In reply to comment #10)
> (In reply to comment #8)
skip
> 
> > here is bugfix and my comments (going out of upstream):
> > visualvm.src: W: invalid-license GPLv2 + Classpath Exception
> >  -This is how upstream has chosen to License it, and it is a widely known
> > license type.
> 
> This license name is invalid. This has nothing to do with upstream. You are
> supposed to use "Short name" column from
> https://fedoraproject.org/wiki/Licensing#Good_Licenses for license tag.
> 

fixed

> > visualvm.src: W: patch-not-applied Patch0: visualvm-debuginfo.patch
> >  -patch is applied, because of it's nature, in build. And IS applied
> 
> Add comment to the spec file explaining this.

fixed 
> 
> > visualvm.x86_64: W: non-etc-or-var-file-marked-as-conffile
> > /usr/lib64/visualvm/etc/visualvm.clusters
> > visualvm.x86_64: W: non-etc-or-var-file-marked-as-conffile
> > /usr/lib64/visualvm/etc/visualvm.conf
> >  -Upstream maintains conf files within %{_libdir}/visualvm/etc/ .. thereis
> > nothing I can change about it
> > 
> > visualvm.x86_64: W: no-documentation
> >  -Upstream does not provide any.
> 

fixed 

> 
> > visualvm.x86_64: E: zero-length
> > /usr/lib64/visualvm/visualvm/config/Modules/org-netbeans-modules-options-keymap.xml_hidden
> > ...
> > visualvm.x86_64: E: zero-length /usr/lib64/visualvm/profiler/.lastModified
> >  -Upstream generates them and it is best not to delete them as upstream may
> > rely on them for something.
> >
> > visualvm.x86_64: W: hidden-file-or-dir
> > /usr/lib64/visualvm/profiler/.lastModified
> >  -Added by upstream build.
> 
> Are they really needed? "Upstream may rely on them for something" is not the
> answer. If you can't figure out if these files are really needed, ask upstream.
> If there is no other way, remove these empty files and re-create them in post
> (remove in postun). Though I very much doubt it would harm anything if they
> were missing.

fixed - upstream rely on them, but dont'really on wether there is first empty
line in them.

> 
> > visualvm.x86_64: W: dangling-symlink /usr/lib64/visualvm
> > /platform/usr/share/netbeans/platform12
> >  -visualvm depends on netbeans and the above target will exist when netbeans is
> > installed. (tested by installation)
> 
> No problem here
> 
> > visualvm.x86_64: W: no-manual-page-for-binary jvisualvm
> >  -Binary launches a gui.
> 
> no problem
> 
> > visualvm.x86_64: W: non-standard-dir-in-usr jvm
> >  -Created by upstream -- not much I can change.
> 
> Really? You are using configure instead of %configure macro and you are not
> setting sysconfdir. I am guessing that's why etc dir ends up in /usr
> 

default prefix is /usr

> > visualvm-debuginfo.x86_64: W: invalid-license GPLv2 + Classpath Exception
> > 3 packages and 0 specfiles checked; 18 errors, 12 warnings.
> > 
> > 
> > visualvm.x86_64: W: conffile-without-noreplace-flag
> > /usr/lib64/visualvm/etc/visualvm.clusters
> > visualvm.x86_64: W: non-etc-or-var-file-marked-as-conffile
> > /usr/lib64/visualvm/etc/visualvm.conf
> > -FIXED
> > 
> > visualvm-debuginfo.x86_64: E: debuginfo-without-sources
> > -FIXED (by upstream and in spec)
> 
> I'll start official review, but you'll have to fix those issues one way or the
> other.

(In reply to comment #11)
> (In reply to comment #10)
> > > visualvm.x86_64: E: zero-length
> > > /usr/lib64/visualvm/visualvm/config/Modules/org-netbeans-modules-options-keymap.xml_hidden
> > > ...
> > > visualvm.x86_64: E: zero-length /usr/lib64/visualvm/profiler/.lastModified
> > >  -Upstream generates them and it is best not to delete them as upstream may
> > > rely on them for something.
> > >
> > > visualvm.x86_64: W: hidden-file-or-dir
> > > /usr/lib64/visualvm/profiler/.lastModified
> > >  -Added by upstream build.
> > 
> > Are they really needed? "Upstream may rely on them for something" is not the
> > answer. If you can't figure out if these files are really needed, ask upstream.
> > If there is no other way, remove these empty files and re-create them in post
> > (remove in postun). Though I very much doubt it would harm anything if they
> > were missing.
> 
> .lastModified files should be created in %post (removed in %postun). See
> http://bits.netbeans.org/dev/javadoc/org-netbeans-bootstrap/overview-summary.html#java.io.File-.lastModified

fixed


Current status:

visualvm.x86_64: W: non-etc-or-var-file-marked-as-conffile
/usr/lib64/visualvm/etc/visualvm.clusters
visualvm.x86_64: W: non-etc-or-var-file-marked-as-conffile
/usr/lib64/visualvm/etc/visualvm.conf
  -installed by upstream. Stil no clue how to fix :(

visualvm.x86_64: W: dangling-symlink /usr/lib64/visualvm/platform
/usr/share/netbeans/platform12
visualvm.x86_64: W: no-manual-page-for-binary jvisualvm
visualvm.x86_64: W: non-standard-dir-in-usr jvm 
  - default prefix is /usr
2 packages and 0 specfiles checked; 0 errors, 5 warnings.

visualvm.src: W: spelling-error Summary(en_US) profiler -> profile, profiles,
profiled
visualvm.src: W: patch-not-applied Patch0: visualvm-debuginfo.patch
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
visualvm.spec: W: patch-not-applied Patch0: visualvm-debuginfo.patch
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
^explained in specfile comment^

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]