Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 216517 - Review Request: gnome-valgrind-session - Run an entire GNOME session under valgrind
Summary: Review Request: gnome-valgrind-session - Run an entire GNOME session under va...
Keywords:
Status: CLOSED DUPLICATE of bug 444113
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-11-20 21:52 UTC by Dave Malcolm
Modified: 2008-04-25 03:24 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-04-25 03:24:06 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Dave Malcolm 2006-11-20 21:52:02 UTC
Spec URL:
https://testing.108.redhat.com/source/browse/*checkout*/testing/trunk/incubator/gnome-valgrind-session/gnome-valgrind-session.spec?rev=292

SRPM URL:
https://testing.108.redhat.com/files/documents/78/162/gnome-valgrind-session-1.0-1.src.rpm

Description: 
This package adds two options to gdm's Session menu to run an entire GNOME
session under valgrind, and gather the results to report files in your home
directory names valgrind-session.0, valgrind-session.1, etc.

The two sessions filter the output, checking either for errors, or for memory
leaks.

Comment 2 Hans Petter Jansson 2006-11-25 15:34:26 UTC
Dave, I put the tarball you asked for at

http://hp.cl.no/proj/gnome-valgrind-session/src/


Comment 3 Parag AN(पराग) 2006-11-26 15:36:52 UTC
Update SPEC file for working Source tarball link.
Tarball contains backup copy of gnome-valgrind-errors-postprocess as
gnome-valgrind-errors-postprocess~. That should be removed.
However it will be good to have License copy included in tarball.

Can you update new SRPM with above changes? Then i will do review of this package.

Comment 4 Parag AN(पराग) 2006-11-26 15:40:05 UTC
You should always give updated SPEC and SRPM links. I saw that you updates SPEC
in comment 2 but where is its SRPM? Now you should submit 
gnome-valgrind-session-1.0-3.src.rpm and its spec.

Comment 5 Parag AN(पराग) 2006-11-29 17:22:16 UTC
ping

Comment 6 Hans Petter Jansson 2006-12-18 20:41:16 UTC
I've put up a new tarball at http://hp.cl.no/proj/gnome-valgrind-session/src/.
It does not contain any backup files, and I added a LICENSE file and individual
license headers to the scripts. The new version is 1.1. I'll leave the FC
packaging fixes to Dave, if he's got the time.


Comment 7 Dave Malcolm 2006-12-18 20:43:14 UTC
Thanks hpj.   

I'm doomed timewise until early January, I hope to pick this up again then.

Comment 8 Parag AN(पराग) 2006-12-19 06:43:29 UTC
I saw tarball
Is single line in LICENSE file, acceptable?
"This package is in the Public Domain."


Comment 9 Hans Petter Jansson 2006-12-19 08:33:34 UTC
I am not a lawyer, so I don't know what the exact requirements are to release
something into the public domain. I would assume that it's sufficient if you can
prove beyond reasonable doubt that I said it is.

Comment 10 Parag AN(पराग) 2006-12-27 05:03:35 UTC
Better to discuss this then on Fedora-extras mailing list.

Comment 11 Parag AN(पराग) 2007-02-22 10:28:33 UTC
Any updates here or is this review DEAD?

Comment 12 Jason Tibbitts 2007-06-05 18:53:33 UTC
Sure looks dead to me.

Comment 13 Matthias Clasen 2007-09-12 13:28:51 UTC
Reopening. Here are some new packages:

http://people.redhat.com/mclasen/gnome-valgrind-session/

Comment 14 Matthias Clasen 2007-09-12 13:29:56 UTC
Wrt to the license question, "Public Domain" is a valid license according to
http://fedoraproject.org/wiki/Licensing#SoftwareLicenses

Comment 15 Debarshi Ray 2007-12-25 16:48:47 UTC
Ping?

Matthias, if you want to "re-open" this review I think you need to submit a
fresh review and mark this as duplicate of that one.

Comment 16 Jason Tibbitts 2008-01-21 17:48:34 UTC
Matthias, you should at least be CC'd on this ticket if you're going to push it
forward.  Otherwise I don't see how you would notice the commentary.

Comment 17 Jason Tibbitts 2008-01-27 20:25:39 UTC
I wasn't successful in getting anyone who is Gnome-savvy to look at this, so
I'll just ask: is anyone still interested in this package?  I will review it if
someone still wants to submit it.

Comment 18 Debarshi Ray 2008-01-27 20:51:18 UTC
I am interested in submitting this, but it will be almost 7 days before I submit
this. Freshening up the anjuta and gnome-build packages will keep me
pre-occupied till then. If someone wants to take it up before that, feel free.

Comment 19 Jason Tibbitts 2008-04-24 21:41:41 UTC
Well, it's been far longer than seven days.  Still interested?  Otherwise I
guess I'll close this again.

Comment 20 Debarshi Ray 2008-04-25 02:37:16 UTC
Oops. It slipped my mind. I am still interested. Will file a new bug and close
this one as a duplicate.

Comment 21 Debarshi Ray 2008-04-25 03:24:06 UTC

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


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