This is my first package, I am looking for sponsors. Thank you very much! Spec URL: http://sites.google.com/site/libertysopenworld/Home/goldendict.spec?attredirects=0&d=1 SRPM URL: http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-0.1.20091111git.fc12.src.rpm?attredirects=0&d=1 Description: Use of WebKit for an accurate articles' representation, complete with all formatting, colors, images and links; Support of multiple dictionary file formats and MediaWiki-based sites; Word suggestion; Full unicode support; Scan popup functionality; Tabbed browsing and more.
Updated package: Now it pass the rpmlint test and conform the guideline much more. spec: http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-1.20091209gitc83b6cd.fc12.src.rpm?attredirects=0&d=1 srpm: http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-1.20091209gitc83b6cd.fc12.src.rpm?attredirects=0&d=1
firstly, make sure you have a FAS account, and join the packager group. secondly, pickup some another package to review, in order to show your understanding to packaging guidelines. there is no sponsors in FZUG community (yet), however I believe many people from FZUG can help to review your package, and we may redirect some sponsors to you. Just keep active! Or you and your package will be forgotten. btw your updated spec link is wrong.
(In reply to comment #1) > Updated package: > Now it pass the rpmlint test and conform the guideline much more. > > spec: > http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-1.20091209gitc83b6cd.fc12.src.rpm?attredirects=0&d=1 > srpm: > http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-1.20091209gitc83b6cd.fc12.src.rpm?attredirects=0&d=1 Sorry the spec file is wrong, here is the spec file: http://sites.google.com/site/libertysopenworld/Home/goldendict.spec?attredirects=0&d=1
(In reply to comment #2) > firstly, make sure you have a FAS account, and join the packager group. > > secondly, pickup some another package to review, in order to show your > understanding to packaging guidelines. > > there is no sponsors in FZUG community (yet), however I believe many people > from FZUG can help to review your package, and we may redirect some sponsors to > you. Just keep active! Or you and your package will be forgotten. > > btw your updated spec link is wrong. Thank you very much for providing me such valuable information. I will try to review some other packages and keep active! I am using fedora for about 4 years and really want to contribute something to the fedora.
I cannot connect to http://site.google.com because of, you know, our Great Fire Wall of China. Please consider to use another service that is not blocked, and is easy to reach (provide a clean URL).
It is hard to find a good host place. How about this: ftp://xth_4709239:123456@ftp.xtreemhost.com/htdocs/goldendict-0.9.0-1.20091209gitc83b6cd.fc12.src.rpm ftp://xth_4709239:123456@ftp.xtreemhost.com/htdocs/goldendict.spec
I can read it now. Please note the bugzilla archive is open to all and open to search, is it OK to show the username and password? The spec is quite well written in my opinion, congratulations! Please fix %doc, at least there should be a COPYING which is the GPL3 text. I've not built it on my own system yet, will try it later. The git clone command is repeated twice.
It is a bad idea to show the ftp user and pass. But I will change my password if I can find out a better place to host the files. I have updated spec file and srpm to include the license file as doc.Also I have changed the group to Application/System according to stardict's spec file. Thank you very much for the advise and encouragement. SPEC: ftp://xth_4709239:123456@ftp.xtreemhost.com/htdocs/goldendict.spec SRPM ftp://xth_4709239:123456@ftp.xtreemhost.com/htdocs/goldendict-0.9.0-2.20091209gitc83b6cd.fc12.src.rpm
Updated package, include latest fixes of punctuation indexing and sound problems. SPEC: http://sites.google.com/site/libertysopenworld/Home/goldendict.spec?attredirects=0&d=1 SRPM http://sites.google.com/site/libertysopenworld/Home/goldendict-0.9.0-3.20091226git7a03248.fc12.src.rpm?attredirects=0&d=1
Spec and srpm links is broken.
I have tried link, it is working.
I have uploaded SRPM here: http://ifile.it/j3ceih5/goldendict-0.9.0-3.20091226git7a03248.fc12.src.rpm and spec file: http://pastebin.com/m29f19a4
The new RPM SRPM http://ifile.it/de6r5ow/goldendict-0.9.0-4.20100206git373d861.fc12.src.rpm SPEC: http://pastebin.com/m39a80056
* Build failure - Your srpm does not build on rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=2036675 Missing BR: phonon-devel This is a successful build with phonon-devel added as BR: https://koji.fedoraproject.org/koji/taskinfo?taskID=2036678
Thank you very much for reporting the problem. The upstream add the requirement phonon but I forgot to add the requirement in SPEC. Here is an updated version SPEC: http://pastebin.com/L5Pf8GFA SRPM: http://ifile.it/an6xvp0/goldendict-0.9.0-5.20100307git83115ad.fc12.src.rpm
Why I always fail to build using koji? Thanks. $ koji build dist-rawhide goldendict-0.9.0-5.20100307git83115ad.fc12.src.rpm .... 2036716 build (dist-rawhide, goldendict-0.9.0-5.20100307git83115ad.fc12.src.rpm): open (ppc10.phx2.fedoraproject.org) -> FAILED: ActionNotAllowed: policy violation
Koji build of the new package: https://koji.fedoraproject.org/koji/taskinfo?taskID=2038695
(In reply to comment #17) > Koji build of the new package: > https://koji.fedoraproject.org/koji/taskinfo?taskID=2038695 Translations are missing in this build.
necleo, Thank you very much for reporting the problems. I have updated the package and add the missing translations. SPEC: http://pastebin.com/5Aqme0Gn SRPM: http://ifile.it/elaz57r/goldendict-0.9.0-6.20100307git83115ad.fc12.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2053625
Interesting package. Some notes: ? git based source - Would you explain why you have taken the source based from git tree instead of the released one? ! BuildRoot - BuildRoot is no longer needed (and simply ignored) https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag * BR - ">= 4.5" on "BR: qt-devel" is not needed because all packages on currently supported Fedora branches all satisfy this packages. ( See the last line of https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires ) ! Idendation - When you divide one command line to several line using backtrash, would you add some identation to make it easier to see? * desktop file - Once you install desktop file by desktop-file-install, desktop-file-validate is no longer needed. - It is better that you use "Icon=goldendict" (i.e. using only file name, without extension). https://fedoraproject.org/wiki/Packaging/Guidelines#Icon_tag_in_Desktop_Files * install usage -------------------------------------------------------------------------- 62 install -dD $RPM_BUILD_ROOT%{_datadir}/apps/goldendict/locale -------------------------------------------------------------------------- - "-D" option is not needed here. * $RPM_BUILD_ROOT vs %buildroot https://fedoraproject.org/wiki/Packaging/Guidelines#Using_.25.7Bbuildroot.7D_and_.25.7Boptflags.7D_vs_.24RPM_BUILD_ROOT_and_.24RPM_OPT_FLAGS - Please choose one and don't use both. * Directory ownership issue https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Common_Mistakes - The following directories themselves are not owned by any packages. --------------------------------------------------------------------------- %{_datadir}/apps/goldendict/ %{_datadir}/apps/goldendict/locale/ --------------------------------------------------------------------------- By the way, would you host your spec / srpm on somewhere from which I can download them directly by $ wget -N ?
Thank you very much for your suggestion. ? git based source There is a stability bug in the 0.9 stable version. I have asked the author about this and the author suggest me to use git version. Also I have tested for months and the git version is more stable than 0.9 version. ! BuildRoot If I delete buildroot line, the rpmlint will give me a warning about the non-presence of buildroot. * BR Right, it is unnecessary. ! Idendation Right, I have added the indentation. * desktop file I have just follow the guide which recommend people to validate it. For the goldendict.png issue, Do I need to use a patch file to fix it? * install usage Yes -D is not useful. * $RPM_BUILD_ROOT vs %buildroot Yes, I have fixed this issue. * Directory ownership issue Thank you for indicating this issue. I have fixed it. New SPEC: http://fantastischmemo.xtreemhost.com/goldendict.spec New SRPM: https://koji.fedoraproject.org/koji/getfile?taskID=2107104&name=goldendict-0.9.0-7.20100307git83115ad.fc14.src.rpm New koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=2107101
I have not checked your latest srpm at all, just replying to your comments for now: (In reply to comment #21) > ! BuildRoot > If I delete buildroot line, the rpmlint will give me a warning about the > non-presence of buildroot. - You can simply ignore this rpmlint warnings. > * desktop file > I have just follow the guide which recommend people to validate it. For the > goldendict.png issue, Do I need to use a patch file to fix it? - You can either use patch or use sed or so.
Okay, as I wrote in the previous comment: - Once desktop-file-install is done, desktop-file-validate is not needed because if desktop file is invalid desktop-file-install also fails. - If is bettter that you use "Icon=goldendict" ! BuildRoot tag is not needed (even if rpmlint warns about this). Then: ------------------------------------------------------------- NOTE: Before being sponsored: This package will be accepted with another few (or no) work. But before I accept this package, someone (I am a candidate) must sponsor you. Once you are sponsored, you have the right to review other submitters' review requests and approve the packages formally. For this reason, the person who want to be sponsored (like you) are required to "show that you have an understanding of the process and of the packaging guidelines" as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Usually there are two ways to show this. A. submit other review requests with enough quality. B. Do a "pre-review" of other person's review request (at the time you are not sponsored, you cannot do a formal review) When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Fedora package collection review requests which are waiting for someone to review can be checked on my wiki page: http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets (Check "No one is reviewing") Review guidelines are described mainly on: http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets ------------------------------------------------------------
Thank you for assigning this RR. I have fixed the icon issue and deleted the unnecessary lines. SPEC: http://fantastischmemo.xtreemhost.com/goldendict.spec SRPM: https://koji.fedoraproject.org/koji/getfile?taskID=2108529&name=goldendict-0.9.0-8.20100307git83115ad.fc14.src.rpm Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=2108528
%{_datadir}/apps is not not owned by any packages in spec. Goldendict should either require kde-filesystem package or be patched to install *.qm into %{_datadir}/%{name}/translations as most Qt programs.
In F12, goldendict is not alone in /usr/share/app. There is a few other apps there. But in my newly installed F13, it is the only one. I will try to look into this problem.
Or add %{?_kde4_macros_api:Requires: kde4-macros(api) = %{_kde4_macros_api} } before the %description section
(In reply to comment #27) > Or add %{?_kde4_macros_api:Requires: kde4-macros(api) = %{_kde4_macros_api} } > before the %description section /usr/share/app is defined as legacy Menu Hierarchies by freedesktop. See http://standards.freedesktop.org/menu-spec/latest/ar01s07.html In fedora, it's owned by kde-filesystem[ %{?_kde4_macros_api:Requires: kde4-macros(api) = %{_kde4_macros_api} } ]
I have added the %{?_kde4_macros_api:Requires: kde4-macros(api) = %{_kde4_macros_api} }. It does not make much different. What does it do? Also the /usr/share/apps directory is used by many other application and goldendict shouldn't own this directory. $ ls /usr/share/apps dcopidlng kdeui kssl goldendict kdewidgets kstyle kabc khtml ktexteditor_docwordcompletion katepart kio_uiserver ktexteditor_insertfile kcertpart kjava ktexteditor_isearch kcm_componentchooser knewstuff ktexteditor_kdatatool kconf_update knotify LICENSES kdbg konqueror proxyscout kdeprint ksgmltools2
Adding "R: kde-filesystem" (or its variant) is sufficient. By the way, I am still waiting for the reply to my comment 23.
If goldendict not KDE application why it should require kde macros? May be would be better to place translations in /usr/share/goldendict or in /usr/share/qt4/translations (%_qt4_translationdir).
So for now R: kde-filesystem is sufficient. kde-filesystem is noarch, just to add some directories and won't pull in any additional dependency.
(In reply to comment #30) > Adding "R: kde-filesystem" (or its variant) is sufficient. > > By the way, I am still waiting for the reply to my comment 23. Some of my reviewed packages. I will try more. 540127 562585 578269
(In reply to comment #33) > (In reply to comment #30) > > Adding "R: kde-filesystem" (or its variant) is sufficient. > > > > By the way, I am still waiting for the reply to my comment 23. > > Some of my reviewed packages. I will try more. > 540127 > 562585 > 578269 Using one of the following Templates to perform a review http://fedoraproject.org/wiki/Category:Package_Maintainers/Review_Template Note: You should understand all items on Review Guidelines before performing a review. http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
(In reply to comment #34) > (In reply to comment #33) > > (In reply to comment #30) > > > Adding "R: kde-filesystem" (or its variant) is sufficient. > > > > > > By the way, I am still waiting for the reply to my comment 23. > > > > Some of my reviewed packages. I will try more. > > 540127 > > 562585 > > 578269 > > Using one of the following Templates to perform a review > > http://fedoraproject.org/wiki/Category:Package_Maintainers/Review_Template > > Note: > You should understand all items on Review Guidelines before performing a > review. > http://fedoraproject.org/wiki/Packaging/ReviewGuidelines Thank you. Here is another two of my reviews: 582974 582931
Well, umm, okay. - Well, bug 582974 is simple, and bug 582931 is .... (srpm itself is) very problematic, however I will appreciate your pre-review. Please add "R: kde-filesystem" when importing this package into Fedora CVS. --------------------------------------------------------- This package (goldendict) is APPROVED by mtasaka --------------------------------------------------------- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Install the Client Tools (Koji)". Now I am sponsoring you. If you want to import this package into Fedora 11/12/13, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me. Removing NEEDSPONSOR.
New Package CVS Request ======================= Package Name: goldendict Short Description: A feature-rich dictionary lookup program Owners: helloworld1 Branches: F-11 F-12 F-13 InitialCC:
CVS done (by process-cvs-requests.py).
goldendict-0.9.0-9.20100307git83115ad.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc13
goldendict-0.9.0-9.20100307git83115ad.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc12
goldendict-0.9.0-9.20100307git83115ad.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc11
goldendict-0.9.0-9.20100307git83115ad.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update goldendict'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc12
goldendict-0.9.0-9.20100307git83115ad.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update goldendict'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc13
goldendict-0.9.0-9.20100307git83115ad.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update goldendict'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/goldendict-0.9.0-9.20100307git83115ad.fc11
goldendict-0.9.0-9.20100307git83115ad.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
goldendict-0.9.0-9.20100307git83115ad.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
goldendict-0.9.0-9.20100307git83115ad.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
Why goldendict placed in Utilities menu? Is Office not right place for it? (I am using KDE)
The goldendict has the same group setting as stardict. Could you find a specific reason to place it under Office?
May be I am wrong but there is Dictionary in 'Additional Category' and suggested 'Related Categories' Office;TextTools. http://standards.freedesktop.org/menu-spec/menu-spec-1.0.html
Please try the 0.9.0-11 version. It has added Dictionary. If it hasn't pushed into stable, please try update-testing
I installed 0.9.0-11 version. GoldenDict is still in Utilities menu because there is Utility in Categories but in freedesktop standards suggested Office;TextTools instead.
Categories=Dictionary;Qt;Utility; 已经有Dictionary了。
The table below describes Additional Categories. The Related Categories column lists one or more categories that are suggested to be used in conjunction with the Additional Category. Note that at least one Main Category must be included in the desktop entry's list of categories. If multiple Main Categories are included in a single desktop entry file, the entry may appear more than once in the menu. If the Related Categories column is blank, the Additional Category can be used with any Main Category. Additional Category: Dictionary Description: A dictionary Related Categories: Office;TextTools (Office is a Main Category) Additional Category: Qt Description: Application based on Qt libraries I think that following to freedestop standards in desktop file should be: Categories=Dictionary;Qt;Office;TextTools;
Look at default gnome-dictionary, it is: Categories=GNOME;GTK;Dictionary;Utility;
(In reply to comment #55) > Look at default gnome-dictionary, it is: > > Categories=GNOME;GTK;Dictionary;Utility; May be wrong Categories there because in packaging guidelines .desktop files MUST follow the freedesktop standards: http://fedoraproject.org/wiki/PackagingGuidelines#Desktop_files
Please don't use this review request which finished its review process almost one month ago for such discuss and instead file a bug against goldendict component then discuss there.
Package Change Request ====================== Package Name: goldendict New Branches: epel7 Owners: moceap helloworld1
Git done (by process-git-requests).