Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
Summary: Review Request: <main package name here> - <short summary here> Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: danken@cs.technion.ac.il QAContact: fedora-package-review@redhat.com
Spec URL: http://ivrix.org.il/redhat/aspell-ar.spec SRPM URL: ivrix.org.il/redhat/aspell-ar-1.2-1.src.rpm Description: Provides the word list/dictionaries for Arabic.
# Note that this package, like other aspell's language packs, does not come up # cleanly through rpmlint, but with the following errors: # E: aspell-he no-binary # E: aspell-he only-non-binary-in-usr-lib # This is because the package contains only data files which sit under /usr/lib. # They have to stay there, as they are architecture-dependent (due to # byte-ordering issues).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
ville.skytta@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: <main |Review Request: aspell-ar - |package name here> - <short |Word list/dictionaries for |summary here> |Arabic
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
petersen@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |varekova@redhat.com
------- Additional Comments From petersen@redhat.com 2006-11-06 01:26 EST ------- Can aspell-ar not be included in Fedora Core?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From danken@cs.technion.ac.il 2006-11-06 02:54 EST ------- I certainly wouldn't mind if aspell-ar (as well as aspell-he) is included in Core.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From notting@redhat.com 2006-11-09 10:58 EST ------- Having it reviewed for Extras is fine to get the review done. I suspect enough things will change around Core + Extras soon enough that we can handle the question of 'where' later.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From kaplanlior@gmail.com 2007-01-01 15:00 EST ------- Any news with this package?
I've already seen it in Debian. I'll be happy to see it in Fedora as well.
Thanks.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From kaplanlior@gmail.com 2007-01-01 15:47 EST ------- I'm pasting a part of a private mail between me and Dan, at his request.
On Mon, Jan 01, 2007 at 09:53:49PM +0200, Lior Kaplan wrote:
About the spec file - why do you manually edit the Makefile instead of running ./configure during %build ?
and in %install $(MAKE) DESTDIR=$(DEB_DESTDIR) install
$(DEB_DESTDIR) has the path for the debian package build root.
I think it will look much more cleaner with these changes.
Suggestions are based on aspell-ar package from Debian (which I maintain with others).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review?
------- Additional Comments From panemade@gmail.com 2007-03-27 03:30 EST ------- rpmlint reported following output on SRPM and RPM
E: aspell-ar no-binary The package should be of the noarch architecture because it doesn't contain any binaries. ==> Ok for aspell packages
E: aspell-ar only-non-binary-in-usr-lib There are only non binary files in /usr/lib so they should be in /usr/share. ==> Ok for aspell packages
W: aspell-ar rpm-buildroot-usage %build DEST_DIR = $RPM_BUILD_ROOT $RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it will break short circuiting.
W: aspell-ar rpm-buildroot-usage %build dictdir = ${RPM_BUILD_ROOT}%{_libdir}/aspell-0.60 $RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it will break short circuiting.
W: aspell-ar rpm-buildroot-usage %build datadir = ${RPM_BUILD_ROOT}%{_libdir}/aspell-0.60 $RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it will break short circuiting.
E: aspell-ar no-cleaning-of-buildroot %install You should clean $RPM_BUILD_ROOT in the %clean section and just after the beginning of %install section. Use "rm -Rf $RPM_BUILD_ROOT".
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |panemade@gmail.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From panemade@gmail.com 2007-03-27 03:41 EST ------- Created an attachment (id=151002) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=151002&action=vie...) Use this modified SPEC and test functionality of Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From danken@cs.technion.ac.il 2007-03-27 07:44 EST ------- It works for me. (Though do note that my functionality test is as good as anyone's. I'm no authority on Arabic spelling.)
-- Memo to self: update aspell-he.spec accordingly.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From panemade@gmail.com 2007-03-27 07:59 EST ------- (In reply to comment #8)
It works for me.
What??
(Though do note that my functionality test is as good as anyone's. I'm no authority on Arabic spelling.)
I mean aspell-ar package is working fine with new changes to SPEC. I have already reviewed few aspell-* packages in Extras which have no problems so I think even I know its functionality test :)
-- Memo to self: update aspell-he.spec accordingly.
What changes you are going to do in aspell-he.spec?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From danken@cs.technion.ac.il 2007-03-27 09:10 EST ------- (In reply to comment #9)
Sorry to be so unclear to warrent two consecutive question marks :-)
It works for me.
What??
I checked aspell-ar with your spec file against a test case that I used before: a page from the Arabic Wikipedia. As expected, aspell recognized Arabic words from the text, and rejected foreign or misspelled words (such as "wikipedia").
Memo to self: update aspell-he.spec accordingly.
What changes you are going to do in aspell-he.spec?
You suggested a couple of cleaning advice to the Arabic spec file. I should apply the same advice to my long-ago-accepted aspell-he package, since its spec file includes all the drawbacks that you pointed. Come to think of it, since I stole the spec file from the French package, maybe it, too, could use a facelift.
Please tell me if/when aspell-ar is approved to enter Extras.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From panemade@gmail.com 2007-03-27 23:22 EST ------- Submit a new package SRPM link with changes I suggested to you and I will do review of it immediately.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From danken@cs.technion.ac.il 2007-03-28 09:46 EST ------- Ok, it's in http://www.ivrix.org.il/redhat/aspell-ar-1.2-2.src.rpm Do note that with your cleaned up spec file, rpmlint of the SRPM cries about configure-without-libdir-spec: the configure script ignores %{_libdir}.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
------- Additional Comments From panemade@gmail.com 2007-03-29 00:01 EST ------- Review: + package builds in mock (development i386). - rpmlint is NOT silent for SRPM and RPM. But following messages are ignorable E: aspell-ar no-binary E: aspell-ar only-non-binary-in-usr-lib E: aspell-ar configure-without-libdir-spec + SPEC file contains explanation about above warnings. + source files match upstream. 154cf762bafdd02db419b62191138738 aspell6-ar-1.2-0.tar.bz2 + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is small; no -doc subpackage required. + %doc does not affect runtime. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + no translations are available. + Dose owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + Requires: aspell >= 12:0.60 + Provides: aspell-ar = 1.2-2 + Not a GUI APP.
APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
danken@cs.technion.ac.il changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From danken@cs.technion.ac.il 2007-03-30 04:41 EST ------- New Package CVS Request ======================= Package Name: aspell-ar Short Description: Arabic word list for Aspell Owners: danken@cs.technion.ac.il Branches: FC-6 devel InitialCC: danken@cs.technion.ac.il
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
wtogami@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
danken@cs.technion.ac.il changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: aspell-ar - Word list/dictionaries for Arabic
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212974
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|normal |medium
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163776 | nThis| |
package-review@lists.fedoraproject.org