[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||thrift-0.9.1-10.fc19 Resolution|--- |ERRATA Last Closed||2014-02-14 03:00:45 --- Comment #62 from Fedora Update System upda...@fedoraproject.org --- thrift-0.9.1-10.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 gil cattaneo punto...@libero.it changed: What|Removed |Added Blocks|652183 (FE-JAVASIG) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=652183 [Bug 652183] Java SIG tracker bug -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 --- Comment #61 from Fedora Update System upda...@fedoraproject.org --- thrift-0.9.1-10.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/thrift-0.9.1-10.fc19 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 Will Benton wi...@redhat.com changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #15 from Will Benton wi...@redhat.com --- Another updated package and spec are available now: http://freevariable.com/thrift/thrift.spec http://freevariable.com/thrift/thrift-0.9.0-4.fc19.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=89Q9evMBs2a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 --- Comment #14 from Will Benton wi...@redhat.com --- OK, a new package and spec are available here: http://freevariable.com/thrift/thrift.spec http://freevariable.com/thrift/thrift-0.9.0-2.fc18.src.rpm Thanks for all of the helpful feedback. Remi, I guess I was confused because the PHP library depends on the binary extension, but it's sorted now. Jonathan, I've not yet enabled D support but will consider doing so after some more investigation. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7eRaYCyGqLa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
Remi Collet fed...@famillecollet.com changed:
What|Removed |Added
CC||fed...@famillecollet.com
--- Comment #8 from Remi Collet fed...@famillecollet.com ---
Small comment about the php sub-package
Please remove
Requires:php
You obviously don't need mod_php and httpd (ABI requirement are enough)
You can also remove
%global php_extdir ...
This macro is defined on fedora and rhel-6 (only required on rhel-5, which
doesn't seems to be targeted)
What is the {php_extdir}/Thrift/ dir used for ?
I also prefer to have the subpackage dependencies in the subpackage definition.
(ex BR: php-devel). It will also save you some conditional stuf for some
language (erlang, ruby) as the sub-package definition already have this
condition. Just make spec a little more legible.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=TYz9l2Jqzma=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
--- Comment #9 from Remi Collet fed...@famillecollet.com ---
%{php_extdir}/Thrift is a pure PHP library, so should be installed (moved) in
the php include_path, so %{_datadir}/php/Thrift (PSR-0 compliant)
According to phpcompatinfo analysis, this library requires :
php(language) = 5.3.0
php-date
php-json
And optionally php-pecl(APC), but seems preferable to not require it in fedora
20 because APC (opcode cache part) have some bad side effects, APCU (dropin
replacment in f20) is fine (no more opcode cache, only user data cache).
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=qtuAP3OkW6a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 MERCIER Jonathan bioinfornat...@gmail.com changed: What|Removed |Added CC||bioinfornat...@gmail.com --- Comment #10 from MERCIER Jonathan bioinfornat...@gmail.com --- Hi, since Fedora support d lang please add this binding the only things to do is: sed -i 's/ldmd/ldmd2/g' configure as configure try to search ldmd whereas the name is ldmd2 thanks -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wWUj8TgDwaa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
--- Comment #11 from MERCIER Jonathan bioinfornat...@gmail.com ---
oh yes you need to set d include dir path.
/configure D_IMPORT_PREFIX=%{_d_includedir}
replace d_lib_flag
sed -i 's/d_lib_flag=.*/d_lib_flag=%{_d_optflags} -Isrc -shared'
lib/d/Makefile
set lib as shared so you need to replace these .a by .so from Makefile at root
project dir
D_EVENT_LIB_NAME = libthriftd-event.a
D_LIB_NAME = libthriftd.a
D_SSL_LIB_NAME = libthriftd-ssl.a
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=ztAZGlb7yxa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
--- Comment #12 from MERCIER Jonathan bioinfornat...@gmail.com ---
Step by step:
$ sed -i 's/d_lib_flags = .*/d_lib_flags = %{_d_optflags} -Isrc -shared/'
lib/d/Makefile
$ sed -i 's/ldmd/ldmd2/g' configure
$ sed -i 's/libthriftd-event.a/libthriftd-event.so/' Makefile
$ sed -i 's/libthriftd.a/libthriftd.so/' Makefile
$ sed -i 's/libthriftd-ssl.a/libthriftd-ssl.so/' Makefile
$ ./configure D_IMPORT_PREFIX=%{_d_includedir}
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=DdknqGaydza=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
--- Comment #13 from MERCIER Jonathan bioinfornat...@gmail.com ---
after a check
fix sed to write expected flags
sed -i s/d_lib_flags = .*/d_lib_flags = $(rpm --eval %{_d_optflags} ) -Isrc
-shared/ lib/d/Makefile
going to bed good luck ^^
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=uzDHVVxz7Ya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 gil cattaneo punto...@libero.it changed: What|Removed |Added CC||punto...@libero.it --- Comment #6 from gil cattaneo punto...@libero.it --- erlang-jsx is available here now https://bugzilla.redhat.com/show_bug.cgi?id=983778 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4KhxMrDaLsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 --- Comment #7 from Will Benton wi...@redhat.com --- Gil, thanks for the great feedback. I'll spin a new package with these fixes (and Erlang support) ASAP. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nk9NIaidkia=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Blocks||652183 (FE-JAVASIG) --- Comment #4 from Peter Lemenkov lemen...@gmail.com --- *** Bug 861783 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=D7VVbW1zWZa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added CC||lemen...@gmail.com --- Comment #5 from Peter Lemenkov lemen...@gmail.com --- Wow! So you're added Erlang - that's nice :) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=NgCPy1beKWa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 Will Benton wi...@redhat.com changed: What|Removed |Added CC||punto...@libero.it --- Comment #2 from Will Benton wi...@redhat.com --- If I read the prior review request correctly, Gil is interested in thrift 0.7.0 because he needs it for a particular project; I need the current version as a dependency for other things. However, I've added him to the CC list here to coordinate if he's interested. (I did adapt his build.xml patch for this package.) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5JrDrm2J8Sa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285
--- Comment #3 from gil cattaneo punto...@libero.it ---
hi
I apologize if I would comment on your work
there are some problems in your spec file:
1 - Requires:gcc-c++ is only used for build c++ libraries and binary files
(!?)
2 - %package -njava-%{name}* should be java-lib%{name} or lib%{name}-java
because it should be the same as the name of the library (should remember
the name of the library is installed or to be installed)
3 - %docdir %{_javadocdir}/%{name} NOT required
%{_javadir}/lib%{name}-%{version}-javadoc.jar NOT required
4 - NO maven files (pom or depmad)
5 - the java library should at least have the following libraries as Requires
mvn(org.slf4j:slf4j-api)
mvn(commons-lang:commons-lang)
mvn(org.apache.httpcomponents:httpclient)
mvn(org.apache.httpcomponents:httpcore)
sorry, but i haven't interest to update the my package at this version
regards
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=tG7EuYcJvga=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 982285] Review Request: thrift - software framework for scalable cross-language services development
https://bugzilla.redhat.com/show_bug.cgi?id=982285 --- Comment #1 from Jason Tibbitts ti...@math.uh.edu --- There's already a thrift review in https://bugzilla.redhat.com/show_bug.cgi?id=861783 Could you two perhaps work together? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MKhmnlEeYBa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
