Bug 177507 - Review Request: pida
Review Request: pida
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Thorsten Leemhuis
Fedora Package Reviews List
:
Depends On:
Blocks: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2006-01-11 06:21 EST by Rudolf Kastl
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-05-22 15:56:06 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)
pida 0.2.2-0.1 spec file (2.60 KB, text/plain)
2006-01-11 06:21 EST, Rudolf Kastl
no flags Details
pida 0.2.2-0.1 src rpm (137.45 KB, application/octet-stream)
2006-01-11 06:23 EST, Rudolf Kastl
no flags Details
pida 0.2.2-0.2 src rpm (137.53 KB, application/octet-stream)
2006-01-11 06:40 EST, Rudolf Kastl
no flags Details
pida 0.2.2-0.2 spec file (2.68 KB, application/octet-stream)
2006-01-11 06:43 EST, Rudolf Kastl
no flags Details
pida 0.3.1-0.1 spec file (3.67 KB, text/plain)
2006-02-08 04:43 EST, Rudolf Kastl
no flags Details
pida 0.3.1-0.2 spec file (3.72 KB, text/plain)
2006-02-08 04:49 EST, Rudolf Kastl
no flags Details

  None (edit)
Description Rudolf Kastl 2006-01-11 06:21:08 EST
the python integrated development application for review.

also looking for a sponsor.
Comment 1 Rudolf Kastl 2006-01-11 06:21:08 EST
Created attachment 123047 [details]
pida 0.2.2-0.1 spec file
Comment 2 Rudolf Kastl 2006-01-11 06:23:04 EST
Created attachment 123049 [details]
pida 0.2.2-0.1 src rpm
Comment 3 Rudolf Kastl 2006-01-11 06:40:30 EST
Created attachment 123051 [details]
pida 0.2.2-0.2 src rpm
Comment 4 Rudolf Kastl 2006-01-11 06:43:15 EST
Created attachment 123052 [details]
pida 0.2.2-0.2 spec file
Comment 5 Rudolf Kastl 2006-01-11 12:53:53 EST
list of known issues:

- gazpacho integration not working with since gazpacho 0.6.3 too new (will be
fixed with pida 0.3.0 in a few weeks.

- usually bicyclerepair would be required to load the python plugins but i left
the dependencys out since the plugins (browser/profiler/debugger) dont load
anyways. (will be fixed with 0.3.0)

Comment 6 Thorsten Leemhuis 2006-01-21 10:48:01 EST
- This
>Requires:   python-abi = %(%{__python} -c "import sys ; print sys.version[:3]")
is not needed anymore -- works automatically in FC4 and later
- Don't repeat the name of the package in the beginning of the summary
- The license doesn't look like GPL
- Description needs a linebreak after 80 chars
- Description might be a bit to long
- Change
Requires:   desktop-file-utils
to
BuildRequires:   desktop-file-utils
- In the future please add versions in the changelog entrys
- why "Release: 0.2" and not "Release: 2"?
- saw those warnings when starting pida:
which: no pydoc in
(/usr/kerberos/bin:/usr/local/bin:/bin:/usr/bin:/usr/X11R6/bin:/home/rpmbuild/usr/bin:/home/rpmbuild/server/usr/bin)
which: no xemacs in
(/usr/kerberos/bin:/usr/local/bin:/bin:/usr/bin:/usr/X11R6/bin:/home/rpmbuild/usr/bin:/home/rpmbuild/server/usr/bin)
Are they relevant?
- In the future please upload the srpms somewhere to the web and post only post
links to it -- don't attach the packages in bugzilla

(In reply to comment #5)
> list of known issues:
> - usually bicyclerepair would be required to load the python plugins but i left
> the dependencys out since the plugins (browser/profiler/debugger) dont load
> anyways. (will be fixed with 0.3.0)

I would add it nevertheless 
Comment 7 Rudolf Kastl 2006-02-08 04:43:56 EST
Created attachment 124368 [details]
pida 0.3.1-0.1 spec file

version upgrade, lots of fixes.
Comment 8 Rudolf Kastl 2006-02-08 04:49:46 EST
Created attachment 124369 [details]
pida 0.3.1-0.2 spec file

only fixes summary as suggested by thl
Comment 9 Rudolf Kastl 2006-02-08 04:57:42 EST
(In reply to comment #6)
> - This
> >Requires:   python-abi = %(%{__python} -c "import sys ; print sys.version[:3]")
> is not needed anymore -- works automatically in FC4 and later

alright thats cleaned up now.

> - Don't repeat the name of the package in the beginning of the summary

cosmetical fix done ;)

> - The license doesn't look like GPL

its MIT... fixed

> - Description needs a linebreak after 80 chars

linebreaks done

> - Description might be a bit to long

gotta recheck that. shouldnt be a big problem to make it shorter

> - Change
> Requires:   desktop-file-utils
> to
> BuildRequires:   desktop-file-utils

thats correct and fixed now

> - In the future please add versions in the changelog 

also done

> - why "Release: 0.2" and not "Release: 2"?

this has multible reasons. first i dont want to give the impression that the
package is yet really production ready and second of all i wanted to start of
with a clean -1 release once its approved. in my eyes pida is very promising
software but very young yet so it can definitely use some love and care already.
0.3.1 is a useable release if the embedded vim editor is used.

> - saw those warnings when starting pida:
> which: no pydoc in
>
(/usr/kerberos/bin:/usr/local/bin:/bin:/usr/bin:/usr/X11R6/bin:/home/rpmbuild/usr/bin:/home/rpmbuild/server/usr/bin)
> which: no xemacs in
>
(/usr/kerberos/bin:/usr/local/bin:/bin:/usr/bin:/usr/X11R6/bin:/home/rpmbuild/usr/bin:/home/rpmbuild/server/usr/bin)
> Are they relevant?

optional components. pida is a framework.

> - In the future please upload the srpms somewhere to the web and post only post
> links to it -- don't attach the packages in bugzilla

alright but unfortunately thats the reason why i dont have the src rpm up yet. 

> 
> (In reply to comment #5)
> > list of known issues:
> > - usually bicyclerepair would be required to load the python plugins but i left
> > the dependencys out since the plugins (browser/profiler/debugger) dont load
> > anyways. (will be fixed with 0.3.0)
> 
> I would add it nevertheless 

seems to be not even required anymore. but i will talk back to upstream. going
to clear up the issue until the next src rpm upload.

TODO:

1) do something with the stuff in docs/ folder... maybe it would be nice to pdf
those html manuals. suggestions?

2) get the vim problem fixed (actually i had problems to get the external vim
editor going)

3) build install and split off the in extras/ included gazpacho plugin

4) report all found issues to upstream
Comment 10 Thorsten Leemhuis 2006-08-06 08:41:16 EDT
ping che -- any progress planed?
Comment 11 Jason Tibbitts 2007-05-07 01:12:25 EDT
It's been a really long time since anything happened with this ticket; what's up?

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