[Bug 487281] Review Request: eGroupware - A web-based groupware suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487281 --- Comment #30 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 2011-04-23 06:55:33 EDT --- I have got answer about estimated dates from Ralf by mail: Hi Pavel, We are not yet finished. Current plan for the release is end of May. After that, we can try to move to extending stock PEAR classes. Ralf -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673784] Rename Request: mingw32-filesystem - mingw-filesystem - Cross compiler base filesystem and environment
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=673784
--- Comment #21 from Erik van Pienbroek erik-fed...@vanpienbroek.nl
2011-04-23 08:26:25 EDT ---
The new packaging guidelines have been approved in the FPC meeting of April 13:
http://meetbot.fedoraproject.org/fedora-meeting/2011-04-13/fpc.2011-04-13-15.59.html
In order to comply with these guidelines, some changes have been applied in the
mingw-filesystem package, so here's a new package which implements everything
mentioned in the packaging guidelines.
Spec URL:
http://svn.openftd.org/svn/fedora_cross/mingw-filesystem/mingw-filesystem.spec
SRPM URL: http://ftd4linux.nl/contrib/mingw-filesystem-68-1.fc15.src.rpm
* Thu Apr 14 2011 Erik van Pienbroek epien...@fedoraproject.org - 68-1
- Make the various macros compliant with the new packaging guidelines:
https://fedorahosted.org/fpc/ticket/71
- Dropped the macro %{_mingw_default_requires} as it isn't needed
anymore with the new guidelines
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673784] Rename Request: mingw32-filesystem - mingw-filesystem - Cross compiler base filesystem and environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673784 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Blocks||673788 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673784] Rename Request: mingw32-filesystem - mingw-filesystem - Cross compiler base filesystem and environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673784 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Blocks||673786 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673786] Rename Request: mingw32-binutils - mingw-binutils - Cross-compiled version of binutils for Win32 and Win64 environments
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673786 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Depends on||673784 Summary|Rename Request: |Rename Request: |mingw32-binutils - |mingw32-binutils - |cross-binutils -|mingw-binutils - |Cross-compiled version of |Cross-compiled version of |binutils for Win32 and |binutils for Win32 and |Win64 environments |Win64 environments --- Comment #2 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-23 08:39:33 EDT --- The FPC has agreed in the meeting of April 13 to approve packaging guidelines where the prefix 'mingw-' will be used for source rpms for MinGW packages (for both Win32 and Win64 targets). Due to this, this package will be renamed to 'mingw-binutils'. Spec URL: http://svn.openftd.org/svn/fedora_cross/mingw-binutils/mingw-binutils.spec SRPM URL: http://ftd4linux.nl/contrib/mingw-binutils-2.21.51.0.8-2.fc15.src.rpm * Thu Apr 14 2011 Erik van Pienbroek epien...@fedoraproject.org - 2.21.51.0.8-2 - Made the package compliant with the new packaging guidelines -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673788] Rename Request: mingw32-gcc - mingw-gcc - GCC Cross-compiler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673788 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Depends on||673784 Summary|Rename Request: mingw32-gcc |Rename Request: mingw32-gcc |- cross-gcc - GCC |- mingw-gcc - GCC |Cross-compiler |Cross-compiler --- Comment #3 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-23 08:36:15 EDT --- The FPC has agreed in the meeting of April 13 to approve packaging guidelines where the prefix 'mingw-' will be used for source rpms for MinGW packages (for both Win32 and Win64 targets). Due to this, this package will be renamed to 'mingw-gcc'. Spec URL: http://svn.openftd.org/svn/fedora_cross/mingw-gcc/mingw-gcc.spec SRPM URL: http://ftd4linux.nl/contrib/mingw-gcc-4.6.1-0.1.20110413.fc15.src.rpm * Thu Apr 14 2011 Erik van Pienbroek epien...@fedoraproject.org - 4.6.1-0.1.20110413 - Update to gcc 4.6.1 20110413 snapshot - Make the package compliant with the new packaging guidelines -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673784] Rename Request: mingw32-filesystem - mingw-filesystem - Cross compiler base filesystem and environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673784 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Blocks||673790 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673790] Rename Request: mingw32-w32api - mingw-headers - Win32/Win64 header files and stubs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673790 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Depends on||673784 --- Comment #2 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-23 08:44:43 EDT --- The new packaging guidelines have been approved in the FPC meeting of April 13: http://meetbot.fedoraproject.org/fedora-meeting/2011-04-13/fpc.2011-04-13-15.59.html In order to comply with these guidelines, some changes have been applied in this package, so here's a new package which implements everything mentioned in the packaging guidelines. Spec URL: http://svn.openftd.org/svn/fedora_cross/mingw-headers/mingw-headers.spec SRPM URL: http://ftd4linux.nl/contrib/mingw-headers-1.0-0.8.20110413.trunk.fc15.src.rpm * Thu Apr 14 2011 Erik van Pienbroek epien...@fedoraproject.org - 1.0-0.8.20110413.trunk - Update to 20110413 snapshot of the trunk branch - Made the package compliant with the new packaging guidelines - Enable the secure API (required for wine-gecko) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673784] Rename Request: mingw32-filesystem - mingw-filesystem - Cross compiler base filesystem and environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673784 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Blocks||673792 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673792] Rename Request: mingw32-runtime - mingw-crt - MinGW Windows cross-compiler runtime
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673792 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Depends on||673784 --- Comment #2 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-23 08:47:07 EDT --- The new packaging guidelines have been approved in the FPC meeting of April 13: http://meetbot.fedoraproject.org/fedora-meeting/2011-04-13/fpc.2011-04-13-15.59.html In order to comply with these guidelines, some changes have been applied in this package, so here's a new package which implements everything mentioned in the packaging guidelines. Spec URL: http://svn.openftd.org/svn/fedora_cross/mingw-crt/mingw-crt.spec SRPM URL: http://ftd4linux.nl/contrib/mingw-crt-1.0-0.4.20110413.trunk.fc15.src.rpm * Thu Apr 14 2011 Erik van Pienbroek epien...@fedoraproject.org - 1.0-0.4.20110413.trunk - Update to 20110413 snapshot of the trunk branch - Made the package compliant with the new packaging guidelines -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 694287] Review Request: openCOLLADA - 3D import and export libraries
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=694287
Hans de Goede hdego...@redhat.com changed:
What|Removed |Added
Flag||fedora-review?
--- Comment #28 from Hans de Goede hdego...@redhat.com 2011-04-23 11:42:50
EDT ---
Full review done, results below:
Good:
=
- rpmlint checks return:
openCOLLADA.src: W: spelling-error Summary(en_US) Collada - Collard
openCOLLADA.src: W: spelling-error %description -l en_US opensource - open
source, open-source, outsource
openCOLLADA.src: W: spelling-error %description -l en_US validator -
lavatorial
openCOLLADA.src: W: spelling-error %description -l en_US xml - XML, ml, x ml
openCOLLADA.src:21: W: macro-in-comment %{name}
openCOLLADA.src:21: W: macro-in-comment %{AGE}
openCOLLADA.src: W: invalid-url Source0: openCOLLADA-svn838.tar.xz
openCOLLADA.x86_64: W: spelling-error Summary(en_US) Collada - Collard
openCOLLADA.x86_64: W: spelling-error %description -l en_US opensource - open
source, open-source, outsource
openCOLLADA.x86_64: W: spelling-error %description -l en_US validator -
lavatorial
openCOLLADA.x86_64: W: spelling-error %description -l en_US xml - XML, ml, x
ml
openCOLLADA.x86_64: W: incoherent-version-in-changelog 0.0-3
['0-3.svn838.fc15', '0-3.svn838']
openCOLLADA-devel.x86_64: W: no-documentation
These can all be ignored.
- package meets naming guidelines
- package meets packaging guidelines
- license (MIT) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files
- post/postun ldconfig ok
Needs work:
===
Ok, I'm going to divide this into 2 lists, one with small things and one with
large things. First the small things:
- rpmlint checks return:
openCOLLADA.src:108: W: mixed-use-of-spaces-and-tabs (spaces: line 9, tab:
line 108)
Please don't use tabs in spec files.
openCOLLADA.x86_64: W: file-not-utf8 /usr/share/doc/openCOLLADA-0/AUTHORS
You will need to convert this to utf8, add the following to %prep:
iconv -f ISO_8859-1 -t utf-8 COLLADAStreamWriter/AUTHORS \
COLLADAStreamWriter/AUTHORS.tmp
touch -r COLLADAStreamWriter/AUTHORS COLLADAStreamWriter/AUTHORS.tmp
mv COLLADAStreamWriter/AUTHORS.tmp COLLADAStreamWriter/AUTHORS
- please drop the following comment (it is the same as Source0, so I see no
use for it):
#Source0:%{name}-svn%{AGE}.tar.xz
- please drop the following (already commented) command from the spec file:
#strip --strip-debug *.so.*
Stripping should *never* be done inside the spec file.
- please add -p to the invocation of cp for the doc files, to keep the original
timestamps. Also why do you cp LICENSE, where as for AUTHORS you reference
it with its full path ? This seems inconsistent.
- please add -k to the invocation of dos2unix to keep the original timestamps
on the files
- Group: Applications/Productivity
Is not really appropriate, please use:
Group: System Environment/Libraries
For the main package instead.
- # copy CHANGES.txt
install -p -m 0644 %{S:1} ./
This belongs in %prep IMHO
- Add -p to invocation of cp for installation of header files to preserve the
timestamps on the files
- devel does not requires base package n-v-r
Please change the:
Requires: %{name} = %{version}
For the -devel sub-package into:
Requires: %{name} = %{version}-%{release}
- I don't like the way the changelog is formatted, here is a copy paste from
another review with similar issues:
Please put a blank line in the changelog before each data-name-version line
(except for the first). And prefix / itemize items below the
data-name-version line with - . IE change:
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-3
Fix typo.
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-2
Mark schemas file as config file
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-1
More spec file fixes
* Thu Sep 23 2010 Elad Alfassa e...@macron.co.il - 0.6-0
Version update, some general spec file fixes.
* Sun Jul 25 2010 Elad Alfassa e...@macron.co.il - 0.3-1
initial build
to:
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-3
- Fix typo.
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-2
- Mark schemas file as config file
* Sun Sep 26 2010 Elad Alfassa el...@doom.co.il - 0.6-1
- More spec file fixes
* Thu Sep 23 2010 Elad Alfassa e...@macron.co.il - 0.6-0
[Bug 687875] Review Request: aarddict - Multi-platform dictionary and offline Wikipedia reader
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=687875
--- Comment #11 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-23
12:42:35 EDT ---
Sorry, in my previous mail I missed out the problem section:
Problem which I have found i.e.,
Un-installation failed to remove directories from %{python_sitelib}/
Nirik told me that it is due to those directories are not owned by the package.
After uninstallation:
1.
aarddict
├── icons
│ ├── hicolor
│ │ ├── 26x26
│ │ │ └── apps
│ │ ├── 40x40
│ │ │ └── apps
│ │ └── 64x64
│ │ └── apps
│ └── Human-O2
│ ├── 16x16
│ │ ├── actions
│ │ ├── emblems
│ │ ├── mimetypes
│ │ ├── places
│ │ └── status
│ ├── 22x22
│ │ ├── actions
│ │ ├── emblems
│ │ ├── mimetypes
│ │ ├── places
│ │ └── status
│ ├── 24x24
│ │ ├── actions
│ │ ├── emblems
│ │ ├── mimetypes
│ │ ├── places
│ │ └── status
│ └── 32x32
│ ├── actions
│ ├── emblems
│ ├── mimetypes
│ ├── places
│ └── status
└── locale
├── el
│ └── LC_MESSAGES
└── ru
└── LC_MESSAGES
.
| 38 directories, 0 files |
.
2.
aarddict-0.9.2-py2.7.egg-info
...
| Totally empty |
...
This is a mail reply I received today from upstream:
//..
You must realize that aarddict works on multiple platforms and is also
packaged for Windows, Mac OS X, Ubuntu and Maemo and is installable
via pip inside or outside of virtualenv. setuptool's idea of where
files should be is very different from Fedora. If aarddict only worked
in Linux and assumed GNOME or KDE I would not package any icons except
for app logo (several icon themes are available as part of standard
desktop) - but this is not the case.
I am most certainly not changing anything in the source code structure
to satisfy Fedora packaging requirements - because it would require
completely changing how application locates it's resources and that is
not something I'm willing to spend time on (even if it had a chance to
work cross-platform). Sorry to disappoint you.
Please note that making an RPM that users can install on Fedora is not
the same as making an RPM that Fedora would accept into distro. As I
mentioned before, the only reason I didn't make one is because Fedora
does not package PyICU and I didn't have time and energy to package
PyICU myself _and_ to setup and maintain yum repo. When/if PyICU
becomes available in Fedora direct aarddict RPM download from
aarddict.org would become a usable option for most users.
...//
Still I am interested to maintain this package, since currently the default
dictionary which comes with fedora requires internet connection. Also, this
software helps to use wikipedia and many others websites to be watched offline.
The only solutions I found now are:
1. https://fedorahosted.org/fpc/ticket/79
or
overlook these warning messages:
//
aarddict.noarch: W: file-not-in-%lang
/usr/lib/python2.7/site-packages/aarddict/locale/el/LC_MESSAGES/aarddict.mo
aarddict.noarch: W: file-not-in-%lang
/usr/lib/python2.7/site-packages/aarddict/locale/ru/LC_MESSAGES/aarddict.mo
..//
The above solution resolves all problems
Waiting for your response.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 636819] Review Request: gnome-exe-thumbnailer - gnome thumbnailer for exe files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636819 --- Comment #45 from Fedora Update System upda...@fedoraproject.org 2011-04-23 16:50:49 EDT --- gnome-exe-thumbnailer-0.7-4.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 636819] Review Request: gnome-exe-thumbnailer - gnome thumbnailer for exe files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636819 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||gnome-exe-thumbnailer-0.7-4 ||.fc14 Resolution||ERRATA Last Closed||2011-04-23 16:50:56 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 699168] New: Review Request: routino - Router for OpenStreetMap Data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: routino - Router for OpenStreetMap Data https://bugzilla.redhat.com/show_bug.cgi?id=699168 Summary: Review Request: routino - Router for OpenStreetMap Data Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ke...@tigcc.ticalc.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://repo.calcforge.org/f16/routino.spec SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-1.fc14.src.rpm Description: Routino is a command-line application for finding a route between two points using the dataset of topographical information collected by OpenStreetMap. It can be used as a routing tool in Marble. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3021403 Testing instructions: 1. Run these commands (WARNING: This takes ~3 hours in total. The download is over 100 MiB (compressed). If you're in a hurry, try something like liechtenstein.osm.bz2. ;-) ): wget http://download.geofabrik.de/osm/europe/austria.osm.bz2 bunzip2 austria.osm.bz2 mkdir ~/.local/share/marble/maps/earth/routino routino-planetsplitter --dir=$HOME/.local/share/marble/maps/earth/routino \ austria.osm 2. In the Marble preferences, in the route planning tab, edit the profiles to disable all the other routing programs (especially the online services), keep only Routino checked. 3. Then use the route planning applet on the left, select 2 places in Austria (or in whatever map section you downloaded and processed in step 1) and a locomotion method and see the route being computed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 699168] Review Request: routino - Router for OpenStreetMap Data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=699168 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added CC||volke...@gmx.at --- Comment #1 from Kevin Kofler ke...@tigcc.ticalc.org 2011-04-23 17:16:37 EDT --- CCing Volker Fröhlich as a potential reviewer for this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 687875] Review Request: aarddict - Multi-platform dictionary and offline Wikipedia reader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=687875 --- Comment #12 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-23 19:37:43 EDT --- Updated files: SPECS :http://prabindatta.fedorapeople.org/SPECS/aarddict.spec SRPMS: http://prabindatta.fedorapeople.org/SRPMS/aarddict-0.9.2-2.fc14.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 692131] Review Request: sflphone - SIP/IAX2 compatible enterprise-class softphone for multi-way conferencing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=692131 --- Comment #4 from PRABIN KUMAR DATTA linux.n@gmail.com 2011-04-23 19:56:47 EDT --- Updated files: Spec URL: http://prabindatta.fedorapeople.org/SPECS/sflphone.spec SRPM URL: http://prabindatta.fedorapeople.org/SRPMS/sflphone-0.9.13-1.fc14.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 699025] Review Request: jchardet - Automatic charset detection algorithm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=699025 Hushan Jia h...@redhat.com changed: What|Removed |Added CC||h...@redhat.com AssignedTo|nob...@fedoraproject.org|h...@redhat.com Flag||fedora-review? --- Comment #1 from Hushan Jia h...@redhat.com 2011-04-23 20:43:08 EDT --- taking this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691027 --- Comment #36 from Fedora Update System upda...@fedoraproject.org 2011-04-23 20:58:28 EDT --- n2n-2.0.1-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/n2n-2.0.1-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691027 --- Comment #37 from Fedora Update System upda...@fedoraproject.org 2011-04-23 21:00:55 EDT --- n2n-2.0.1-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/n2n-2.0.1-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 692131] Review Request: sflphone - SIP/IAX2 compatible enterprise-class softphone for multi-way conferencing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=692131 --- Comment #5 from nucleo alekc...@googlemail.com 2011-04-23 22:10:35 EDT --- Hi, sflphone uses bundled versions of iax2 and pjproject. See https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries Is it possible to build it against system libax? Looks like bundled pjproject version is very old 0.5.10.3. New releases available here http://www.pjsip.org/release/ If sflphone can be built against recent pjproject releases then maybe it is makes sense to make independent package for pjproject. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 692131] Review Request: sflphone - SIP/IAX2 compatible enterprise-class softphone for multi-way conferencing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=692131 --- Comment #6 from nucleo alekc...@googlemail.com 2011-04-23 22:13:59 EDT --- I was wrong about bundled pjproject version. PJ_VERSION = 1.5.5 used in sflphone-0.9.13 sources. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691096] Review Request: iperf3 - Measurement tool for TCP/UDP bandwidth performance
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691096 Balaji G balaji...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Balaji G balaji...@gmail.com 2011-04-23 23:21:02 EDT --- New Package SCM Request === Package Name: iperf3 Short Description: Measurement tool for TCP/UDP bandwidth performance Owners: balajig8 kevin Branches: f14 f15 e16 InitialCC: balajig8 kevin -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672395] Review Request: eigen3 - A lightweight C++ template library for vector and matrix math
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=672395 Bug 672395 depends on bug 691133, which changed state. Bug 691133 Summary: internal compiler error: in convert_move, at expr.c:324 https://bugzilla.redhat.com/show_bug.cgi?id=691133 What|Old Value |New Value Resolution||CURRENTRELEASE Status|NEW |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693119] Review Request: drupal6-login_destination - Customize login landing page in Drupal 6
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693119 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-04-24 00:10:05 EDT --- drupal6-login_destination-2.12-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693119] Review Request: drupal6-login_destination - Customize login landing page in Drupal 6
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693119 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||drupal6-login_destination-2 ||.12-1.fc15 Resolution||ERRATA Last Closed||2011-04-24 00:10:10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672395] Review Request: eigen3 - A lightweight C++ template library for vector and matrix math
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=672395 --- Comment #7 from Rich Mattes richmat...@gmail.com 2011-04-24 00:27:57 EDT --- Spec URL: http://rmattes.fedorapeople.org/RPMS/eigen3/eigen3.spec SRPM URL: http://rmattes.fedorapeople.org/RPMS/eigen3/eigen3-3.0.0-2.fc15.src.rpm rpmlint: eigen3.spec: W: invalid-url Source0: eigen-3.0.0.tar.bz2 eigen3-devel.noarch: E: zero-length /usr/include/eigen3/Eigen/src/Sparse/SparseAssign.h 1 packages and 1 specfiles checked; 1 errors, 1 warnings. The invalid source URL is because of the tarball rename. The zero-length file was addressed in the first comment: I can remove it, but I would have to patch some other files to get rid of references to it. I checked and the bug that was blocking this bug is no longer an issue as you pointed out. I went ahead and closed it, so this review can continue. Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3022117 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 593559] Review Request: protobuf-c - C bindings for Google's Protocol Buffers
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=593559
--- Comment #24 from David Robinson zxvdr...@gmail.com 2011-04-24 01:38:50
EDT ---
Fixed.
Spec URL: http://zxvdr.fedorapeople.org/RPMS/protobuf-c/protobuf-c.spec
SRPM URL:
http://zxvdr.fedorapeople.org/RPMS/protobuf-c/protobuf-c-0.15-2.fc14.src.rpm
--- protobuf-c.spec.orig 2011-04-21 19:23:06.0 +1000
+++ protobuf-c.spec 2011-04-24 15:29:36.060990432 +1000
@@ -1,6 +1,6 @@
Name: protobuf-c
Version:0.15
-Release:1%{?dist}
+Release:2%{?dist}
Summary:C bindings for Google's Protocol Buffers
Group: System Environment/Libraries
@@ -32,7 +32,9 @@
%build
%configure --disable-static
-make %{?_smp_mflags}
+# Causes build to fail
+#make %{?_smp_mflags}
+make
%check
make check
@@ -52,11 +54,15 @@
%files devel
%defattr(-,root,root,-)
+%dir %{_includedir}/google
%{_includedir}/google/protobuf-c
%{_libdir}/libprotobuf-c.so
%{_libdir}/pkgconfig/libprotobuf-c.pc
%changelog
+* Sun Apr 24 2011 David Robinson zxvdr...@gmail.com - 0.15-2
+- Spec file cleanup
+
* Wed Apr 20 2011 David Robinson zxvdr...@gmail.com - 0.15-1
- New upstream release
- Spec file cleanup
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
