Bug 209870 - Prozilla - Advanced Linux download manager
Prozilla - Advanced Linux download manager
Product: Fedora
Classification: Fedora
Component: prozilla (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Chitlesh GOORAH
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2006-10-07 02:25 EDT by kushaldas@gmail.com
Modified: 2007-11-30 17:11 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-01-02 07:23:49 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
prozilla.spec (5.23 KB, text/plain)
2006-12-09 21:42 EST, Chitlesh GOORAH
no flags Details

  None (edit)
Description kushaldas@gmail.com 2006-10-07 02:25:51 EDT
Review request for a new release.
Spec URL:  http://freeshell.in/~kushal/prozilla.spec
SRPM URL:  http://freeshell.in/~kushal/prozilla-2.0.4-1.src.rpm

rpmlint output :
[kd@kushal redhat]$ rpmlint ./RPMS/i386/prozilla-2.0.4-1.i386.rpm
W: prozilla hidden-file-or-dir /usr/share/locale/ro/LC_MESSAGES/.mo
W: prozilla hidden-file-or-dir /usr/share/locale/it/LC_MESSAGES/.mo
W: prozilla hidden-file-or-dir /usr/share/locale/pt_BR/LC_MESSAGES/.mo
W: prozilla hidden-file-or-dir /usr/share/locale/nl/LC_MESSAGES/.mo
Comment 1 kushaldas@gmail.com 2006-10-07 02:34:21 EDT
typo fixed
Comment 2 Chitlesh GOORAH 2006-10-09 11:32:00 EDT
You can't assign a review to yourself if you are the bug reporter :p

I'll do the reviewer !
Comment 3 Chitlesh GOORAH 2006-10-09 11:52:10 EDT
#1 : BuildRoot is incorrect !

#2 : %makeinstall macro should not be used !

#3 don't ship %{_libdir}/libprozilla.la and %{_libdir}/libprozilla.a. Delete
them during %install

#4 don't shipt#No idea why these files are here..
%ghost /usr/share/locale/it/LC_MESSAGES/.mo
%ghost /usr/share/locale/nl/LC_MESSAGES/.mo
%ghost /usr/share/locale/pt_BR/LC_MESSAGES/.mo
%ghost /usr/share/locale/ro/LC_MESSAGES/.mo

Delete them during %install

#5 no need to ship README twice, but you should rather consider shipping FAQ as

#6 is there a reason why you are using 
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig ?
Comment 4 Frank Arnold 2006-10-09 18:33:44 EDT
(In reply to comment #3)
> #4 don't shipt#No idea why these files are here..
> %ghost /usr/share/locale/it/LC_MESSAGES/.mo
> %ghost /usr/share/locale/nl/LC_MESSAGES/.mo
> %ghost /usr/share/locale/pt_BR/LC_MESSAGES/.mo
> %ghost /usr/share/locale/ro/LC_MESSAGES/.mo
> Delete them during %install

These files almost certainly contain some translation data. They are not 
hidden, instead they just lack a file prefix. I suspect there's something wrong 
with the Makefile or related things. Because these are only 4 languages, this 
must be the translation for libprozilla (I guess the GUI translation works.)

I can't help you much further, sitting in front of Win2000 Box - just pushing 
into the right direction...
Comment 5 kushaldas@gmail.com 2006-10-10 03:50:22 EDT
Hope all fixed :)
Spec URL:  http://freeshell.in/~kushal/prozilla.spec
SRPM URL:  http://freeshell.in/~kushal/prozilla-2.0.4-2.src.rpm

Comment 6 Chitlesh GOORAH 2006-10-11 04:08:06 EDT
Where have the translations gone ?
Comment 7 kushaldas@gmail.com 2006-10-15 05:36:38 EDT
No idea.
I think that is a bug in the makefile.
Comment 8 Chitlesh GOORAH 2006-10-15 06:00:28 EDT
Read Handling Locale Files:
Comment 9 kushaldas@gmail.com 2006-10-15 07:02:11 EDT
No locale files found after installation.
Comment 10 Chitlesh GOORAH 2006-11-04 06:42:07 EST
Yes, there are !!!

chitlesh(SOURCES)[0]$ls prozilla-2.0.4/po
cat-id-tbl.c  es.po   fr.po   it.po           nl.gmo  POTFILES.in  proz.pot  
pt_BR.po  ro.po
ChangeLog     fr.gmo  it.gmo  Makefile.in.in  nl.po   prozgui.pot  pt_BR.gmo 
ro.gmo    stamp-cat-id

Have a look at what you have changed from 2.0.4-1 to 2.0.4-2 !
Comment 11 Chitlesh GOORAH 2006-11-05 05:54:25 EST
Use gettext as BR

%find_lang proz in %install

%files -f proz.lang instead of %files

replace %dir %{_includedir}/prozilla/* by %{_includedir}/prozilla/
Comment 12 kushaldas@gmail.com 2006-11-12 15:00:32 EST
Doing that comes to an error

No translations found for proz in /var/tmp/prozilla-2.0.4-3-root-root
error: Bad exit status from /var/tmp/rpm-tmp.58160 (%install)

Coz the makefile installing them to the real /usr/share without any name ..

installing pt_BR.gmo as /usr/share/locale/pt_BR/LC_MESSAGES/.mo
installing ro.gmo as /usr/share/locale/ro/LC_MESSAGES/.mo
installing nl.gmo as /usr/share/locale/nl/LC_MESSAGES/.mo
installing it.gmo as /usr/share/locale/it/LC_MESSAGES/.mo
Comment 13 Chitlesh GOORAH 2006-12-09 09:41:42 EST
Ok, ill have a look at it.
I feel you are having problems with it, do you mind that I co-package prozilla
with you in FE ?
Comment 14 Chitlesh GOORAH 2006-12-09 21:42:55 EST
Created attachment 143231 [details]
Comment 15 Chitlesh GOORAH 2006-12-09 21:44:31 EST
the makeinstall macro was causing those problems.
the localedir wasn't correctly assigned on the po folders.

Use my spec file (attached)
Comment 16 kushaldas@gmail.com 2006-12-11 23:58:28 EST
(In reply to comment #13)
> Ok, ill have a look at it.
> I feel you are having problems with it, do you mind that I co-package prozilla
> with you in FE ?
You are welcome :)
Sorry for the late reply
Comment 17 Chitlesh GOORAH 2007-01-02 07:23:49 EST
I've committed the new changes to the already available prozilla package in FE.
The builds were successful on FC-5,FC-6 and devel.

I'll invite you to have a look at the spec file.

I'm now closing this bug.

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