[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium --- Additional Comments From [EMAIL PROTECTED] 2007-03-27 04:45 EST --- Thanks for the review! Imported and build, closing. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 --- Additional Comments From [EMAIL PROTECTED] 2007-03-27 05:22 EST --- Can I close this? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-03-27 07:39 EST --- (In reply to comment #11) Can I close this? My bad, when I wrote: closing I should have actually closed it, done now. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-03-25 08:13 EST --- New Package CVS Request === Package Name: paragui Short Description: Graphical User Interface based on SDL Owners:[EMAIL PROTECTED] Branches: FC-6 devel InitialCC: empty -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-03-25 13:43 EST --- cvs done -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 --- Additional Comments From [EMAIL PROTECTED] 2007-03-24 05:58 EST --- (In reply to comment #3) ? Support bindings - It seems that python/ruby bindings are supported. However, when I tried to enable them, it failed due to some error... And I am not familiar with swig... I've enabled the python bindings and fixed the error. Since I know nothing of ruby I have left them disabled for now. ? automake .m4 files - I don't know if any conclusion was made about this, however should -devel package require automake for .m4 file? I don't believe there was a conclusion. But since all used directories must be owned this is the only solution (until some other package the automake becomes the owner) All other items fixed, new version here: Spec URL: http://people.atrpms.net/~hdegoede/paragui.spec SRPM URL: http://people.atrpms.net/~hdegoede/paragui-1.0.4-2.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140
--- Additional Comments From [EMAIL PROTECTED] 2007-03-24 07:53 EST ---
Created an attachment (id=150825)
-- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150825action=view)
mock build log of paragui-1.0.4-2 on FC-devel i386
* Mock build failed on FC-devel i386 as attached.
-
%{__sed} -i.dir -e 's|/src|/src/.libs|' bindings/python/Makefile.in
-
works, however you may like to create a patch.
* I use FC-devel (so python is 2.5), however, on FC-6 or FC-5
python version is 2.4.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140
--- Additional Comments From [EMAIL PROTECTED] 2007-03-24 19:26 EST ---
(In reply to comment #5)
Created an attachment (id=150825)
--
(https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150825action=view)
[edit]
mock build log of paragui-1.0.4-2 on FC-devel i386
* Mock build failed on FC-devel i386 as attached.
-
%{__sed} -i.dir -e 's|/src|/src/.libs|' bindings/python/Makefile.in
-
works, however you may like to create a patch.
Fixed (added to vegastrike-python.patch)
* I use FC-devel (so python is 2.5), however, on FC-6 or FC-5
python version is 2.4.
Good point, I'll adjust the patch + specfile when copying the imported version
over from devel to FC-6
New version here:
Spec URL: http://people.atrpms.net/~hdegoede/paragui.spec
SRPM URL: http://people.atrpms.net/~hdegoede/paragui-1.0.4-3.fc7.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-03-25 01:59 EST --- Okay, all what are addressed are fixed. --- This package (paragui) is APPROVED by me. --- -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 --- Additional Comments From [EMAIL PROTECTED] 2007-03-23 09:59 EST --- Hint to self: fix: /usr/share/aclocal/paragui.m4:12: warning: underquoted definition of AM_PATH_PARAGUI -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140
--- Additional Comments From [EMAIL PROTECTED] 2007-03-23 14:22 EST ---
Well, for 1.0.4-1:
?: Question
*: Must/Should fix
? Support bindings
- It seems that python/ruby bindings are supported.
However, when I tried to enable them, it failed due to
some error...
And I am not familiar with swig...
* Autotool
--
+ make
cd . /builddir/build/BUILD/paragui-1.0.4/missing automake-1.4 --foreign
--include-deps Makefile
WARNING: `automake-1.4' is missing on your system. You should only need it if
you modified `Makefile.am', `acinclude.m4' or `configure.ac'.
You might want to install the `Automake' and `Perl' packages.
Grab them from any GNU archive site.
cd . \
CONFIG_FILES=Makefile CONFIG_HEADERS= /bin/sh ./config.status
--
- Having autotools called after configure is wrong and
must be fixed.
Perhaps timestamps on some files are wrong.
* Timestamps
- Please keep timestamps on the files which are not
created or modified on build stage
(i.e. which are contained in the source tarball and installed
without any modification).
Normally these are
- Text files such as documentations, header files
- image files
- etc..
For this package, the following works.
--
make install DESTDIR=$RPM_BUILD_ROOT INSTALL=%{__install} -p
--
? automake .m4 files
- I don't know if any conclusion was made about this, however
should -devel package require automake for .m4 file?
* URL
- And fix URL.
My firefox shows that when I tries to download src.tarball from
http://www.bms-austria.com/projects/paragui/modules.php?op=modloadname=Downloadsfile=indexreq=viewdownloadcid=1
it is redirected to gnu.org.
* rpmlint
---
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Set_Char_Size
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Load_Glyph
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Done_FreeType
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Get_Kerning
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Init_FreeType
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_New_Memory_Face
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Get_Char_Index
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0
FT_Done_Face
W: paragui undefined-non-weak-symbol /usr/lib/libparagui-1.0.so.0.4.0 FT_MulFix
W: paragui unused-direct-shlib-dependency /usr/lib/libparagui-1.0.so.0.4.0
/lib/libpthread.so.0
W: paragui unused-direct-shlib-dependency /usr/lib/libparagui-1.0.so.0.4.0
/usr/lib/libpng12.so.0
W: paragui unused-direct-shlib-dependency /usr/lib/libparagui-1.0.so.0.4.0
/usr/lib/libtiff.so.3
W: paragui unused-direct-shlib-dependency /usr/lib/libparagui-1.0.so.0.4.0
/usr/lib/libjpeg.so.62
W: paragui unused-direct-shlib-dependency /usr/lib/libparagui-1.0.so.0.4.0
/lib/libgcc_s.so.1
W: paragui-devel no-documentation
---
- unused-direct-shlib-dependency can be ignored.
However, undefined-non-weak-symbol cannot be ignored because
this rpm provides -devel subpackage and linkage against libparagui-1.0
fails due to this symbols.
* Requires
---
[EMAIL PROTECTED] ~]$ pkg-config --libs paragui
-lparagui -lSDL -lpthread -lSDL_image -lexpat -lfreetype -lstdc++
[EMAIL PROTECTED] ~]$ pkg-config --cflags paragui
-D_GNU_SOURCE=1 -D_REENTRANT -I/usr/include/freetype2 -I/usr/include/SDL
---
- This means that -devel package should require SDL-devel, SDL_image-devel
(and perhaps this is wrong).
NOTE: if it is correct that -lSDL or -I/usr/include/SDL is included,
then paragui.pc should have Requires: sdl (i.e. paragui.pc
should require sdl.pc).
And.. anyway paragui.pc should not have explicit
-lwhich are in other packages entry. This entry usually means
that linkage against libparagui-1.0 is incorrect.
I would appreciate it if you would review my review request for
mecab-ipadic (bug 231486), which is very similar with mecab-jumandic.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||233142 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233140] Review Request: paragui - Graphical User Interface based on SDL
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: paragui - Graphical User Interface based on SDL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233140 --- Additional Comments From [EMAIL PROTECTED] 2007-03-20 14:21 EST --- Just a NOTE: SourceURL is found on: http://download.savannah.gnu.org/releases/paragui/ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
