[Bug 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2014-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-11-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Igor Gnatenko i.gnatenko.br...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2013-11-28 04:30:07



--- Comment #27 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
I submitted new review. If anyone want to maintain - I can add as
co-maintainer.

*** This bug has been marked as a duplicate of bug 1035661 ***

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-11-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681



--- Comment #25 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
I want to get this package in Fedora.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-11-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681



--- Comment #26 from Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl ---
(In reply to Igor Gnatenko from comment #25)
 I want to get this package in Fedora.
That'd be nice. Looking at the lack of recent responses from other people on
this bug, I think you should just take over the review request, and maybe add
other people as co-maintainers if they ever get sponsored.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-09-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Pavel Sedlák psed...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
 CC||psed...@redhat.com



-- 
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=tQqWSVMskma=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-09-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Muayyad Alsadi als...@ojuba.org changed:

   What|Removed |Added

 CC||als...@ojuba.org
  Flags|needinfo?(alapshin@fastmail |
   |.fm)|



--- Comment #22 from Muayyad Alsadi als...@ojuba.org ---
I've updated .spec to the latest upstream and fixed the issues

.spec file

http://pastebin.com/raw.php?i=fqgu34Pv
https://docs.google.com/file/d/0B15QfS_FMZ1GcEs2X2xZSGE3RHc/edit?usp=sharing


the .src.rpm file

https://docs.google.com/file/d/0B15QfS_FMZ1GVWhlRFFwclRRakk/edit?usp=sharing

-- 
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=Zq10w7tjuna=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2013-09-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Muayyad Alsadi als...@ojuba.org changed:

   What|Removed |Added

 Attachment #559690|0   |1
is obsolete||
 Attachment #559691|0   |1
is obsolete||



--- Comment #23 from Muayyad Alsadi als...@ojuba.org ---
Created attachment 792723
  -- https://bugzilla.redhat.com/attachment.cgi?id=792723action=edit
luajit-2.0.2.spec

-- 
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=YRRFq61GqNa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-09-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

--- Comment #21 from M. Edward (Ed) Borasky zn...@znmeb.net ---
I don't know if I'm quite ready to take over this package, but it's required by
a package that I do want to use (LuaAV), so I'll certainly take a look at it.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-08-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=718681

Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

  Flags||needinfo?(alaps...@gmx.com)

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-03-24 Thread bugzilla
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=718681

--- Comment #20 from Michael Schwendt mschwe...@gmail.com 2012-03-24 12:47:51 
EDT ---
A brief but hopefully helpful look at the spec file:


 %package devel
 Summary:Development files for %{name}
 Group:  System Environment/Libraries

Library -devel packages typically are in group Development/Libraries whereas
System Environment/Libraries is for the base library packages.

 Requires:   %{name} = %{version}-%{release}

https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

 Requires:   pkgconfig

This can be removed because it is automatic. Take a look at the built rpms with
rpm -qpR ...


 %package static
 Summary:Static library for %{name}
 Group:  System Environment/Libraries
 Requires:   %{name} = %{version}-%{release}

Same here as above, plus: It makes no sense for the -static package to require
the base package. If at all, it could require the -devel package.


 %post devel -p /sbin/ldconfig

 %postun devel -p /sbin/ldconfig

The are not needed for the -devel package. There is nothing in the -devel
package that would be affected by running ldconfig. This is library base
package stuff.


 %files
 %defattr(-,root,root,-)

This %defattr is the default and need not be specified anymore:
https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions


 %{_datadir}/%{name}-%{version}/jit/*
 %dir %{_datadir}/%{name}-%{version}
 %dir %{_datadir}/%{name}-%{version}/jit

Strange order of lines. Due to the '*' wildcard, you could reduce these three
lines to just

  %{_datadir}/%{name}-%{version}/

to include that directory and everything in it properly.


 %dir %{_libdir}/lua
 %dir %{_libdir}/lua/5.1
 %dir %{_datadir}/lua
 %dir %{_datadir}/lua/5.1

Empty directories so far. Intentional? If so, a comment in the spec file would
be appropriate.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-02-06 Thread bugzilla
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=718681

--- Comment #18 from Andrei Lapshin alaps...@gmx.com 2012-02-06 11:54:12 EST 
---
Created attachment 559691
  -- https://bugzilla.redhat.com/attachment.cgi?id=559691
Spec diff

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-02-06 Thread bugzilla
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=718681

Andrei Lapshin alaps...@gmx.com changed:

   What|Removed |Added

 Attachment #527109|0   |1
is obsolete||
 Attachment #527110|0   |1
is obsolete||
 Attachment #527111|0   |1
is obsolete||

--- Comment #17 from Andrei Lapshin alaps...@gmx.com 2012-02-06 11:53:03 EST 
---
Created attachment 559690
  -- https://bugzilla.redhat.com/attachment.cgi?id=559690
Latest spec file

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2012-02-06 Thread bugzilla
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=718681

--- Comment #19 from Andrei Lapshin alaps...@gmx.com 2012-02-06 11:57:17 EST 
---
SRPM:  http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.4.beta9.fc17.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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-12-13 Thread bugzilla
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=718681

Scott Tsai scottt...@gmail.com changed:

   What|Removed |Added

 CC||scottt...@gmail.com

--- Comment #16 from Scott Tsai scottt...@gmail.com 2011-12-13 19:18:38 EST 
---
Informal review. I also can't sponsor people in Fedora.

1. Specify URL of Patch0 like this:
- Patch0: beta8_hotfix1.patch
+ Patch0: http://luajit.org/download/beta8_hotfix1.patch
- Source1:http://luajit.org/download/beta8_hotfix1.patch

2. Rename /usr/bin/luajit-2.0.0-beta8 to /usr/bin/luajit
The main command is only installed as:
 /usr/bin/luajit-2.0.0-beta8
having the 'luajit' binary name change from under the user as
we update to newer beta releases seems clearly undesirable.

Quoting from upstream http://luajit.org/install.html:
to avoid overwriting a previous version,
the beta test releases only install the LuaJIT executable under the versioned
name
(i.e. luajit-2.0.0-beta8). You probably want to create a symlink for
convenience ...

Since we're not packaging multiple versions of luajit in Fedora at this time,
I recommend you simply rename the binary as /usr/bin/luajit
This would also get rid of the no-manual-page-for-binary rpmlint warning. 

3. Make up your mind on whether to support older rpmb versions
If support for EPEL5 is desired, then the -devel package should require
pkgconfig.
Otherwise we can rely on newer rpm features and remove BuildRoot and rm -rf
%{buildroot} from the spec.

Other comments:
I ran the small collection of Lua 5.1 tests from http://testmore.luaforge.net/
and
they worked.

Regarding libluajit calling exit, I did a quick scan of the source and besides
implementing
os.exit() luajit may also call exit() during exception handling errors
(lj_err.c).
rpmlint complaining about shared libraries calling exit seems to be codifying a
policy
that's not universally applicable in the wrong place anyway.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #9 from Andrei Lapshin alaps...@gmx.com 2011-10-09 10:32:41 EDT 
---
Created attachment 527096
  -- https://bugzilla.redhat.com/attachment.cgi?id=527096
Latest spec file

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #8 from Andrei Lapshin alaps...@gmx.com 2011-10-09 10:31:48 EDT 
---
Created attachment 527095
  -- https://bugzilla.redhat.com/attachment.cgi?id=527095
Spec file diff

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #10 from Andrei Lapshin alaps...@gmx.com 2011-10-09 10:40:37 EDT 
---
Latest SRPM
http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.3.beta8.fc17.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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #11 from Andrei Lapshin alaps...@gmx.com 2011-10-09 10:48:28 EDT 
---
Fixed errors pointed out by john5342.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

Andrei Lapshin alaps...@gmx.com changed:

   What|Removed |Added

 Attachment #527095|0   |1
is obsolete||
 Attachment #527096|0   |1
is obsolete||

--- Comment #12 from Andrei Lapshin alaps...@gmx.com 2011-10-09 13:04:56 EDT 
---
Created attachment 527109
  -- https://bugzilla.redhat.com/attachment.cgi?id=527109
Spec file diff with fixed typo

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #13 from Andrei Lapshin alaps...@gmx.com 2011-10-09 13:06:07 EDT 
---
Created attachment 527110
  -- https://bugzilla.redhat.com/attachment.cgi?id=527110
Latest spec file

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #14 from Andrei Lapshin alaps...@gmx.com 2011-10-09 13:07:57 EDT 
---
Created attachment 527111
  -- https://bugzilla.redhat.com/attachment.cgi?id=527111
Spec file diff from previous version

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-10-09 Thread bugzilla
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=718681

--- Comment #15 from Andrei Lapshin alaps...@gmx.com 2011-10-09 13:10:55 EDT 
---
Sorry, found typo in %build section. Fixed and rebuild. New SRPM available by
same address
http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.3.beta8.fc17.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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-09-27 Thread bugzilla
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=718681

--- Comment #7 from john5342 john5...@gmail.com 2011-09-27 12:16:48 EDT ---
Apologies. I can't do the review. Didn't spot the FE-NEEDSPONSOR but the above
points still apply.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-09-27 Thread bugzilla
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=718681

--- Comment #6 from john5342 john5...@gmail.com 2011-09-27 12:15:42 EDT ---
Just a few initial notes looking at the spec file:


The actual shared library (libluajit-5.1.so.2.*) is in the main package as it
should be but libluajit-5.1.so should only be used for linking against and
should therefore be in the devel package.


The buildroot tag is not one of the standard ones so far as i remember but
either way it has not been needed for some time now and is ignored on our build
system.


The hotfix patch (which is obviously upstream) should be provided as a full URL
just like the source package.


%{optflags} must always be used. Adding CFLAGS=%{optflags} to the build
commandline should do the trick according to the in Makefile documentation.


All build output should be in verbose mode to aid in debugging failed builds as
well as verifying that correct flags are used. I don't know of an easy way with
these hand written Makefiles but it certainly should be investigated.


A quick build and rpmlint of the resulting output:
$ rpmlint luajit-*.rpm
luajit.x86_64: W: shared-lib-calls-exit /usr/lib64/libluajit-5.1.so.2.0.0
exit@GLIBC_2.2.5

Can probably be ignored since it is required to implement os.exit()

luajit.x86_64: W: no-manual-page-for-binary luajit-2.0.0-beta8

Not ideal but upstream don't provide one.

luajit-debuginfo.x86_64: E: empty-debuginfo-package

This is an error. Not sure if it is because it's failing to pick up the debug
options in %{optflags} or something else but difficult to tell without knowing
what commands are run.

luajit-devel.x86_64: W: no-documentation

devel requires the main package which does contain docs

luajit-static.x86_64: W: spelling-error %description -l en_US libluajit -
liberality

Ignore

luajit-static.x86_64: W: no-documentation

Same as above

5 packages and 0 specfiles checked; 1 errors, 5 warnings.



If these things are sorted i can probably do a proper review when i get some
time.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-08-30 Thread bugzilla
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=718681

anish apa...@redhat.com changed:

   What|Removed |Added

 CC||apa...@redhat.com

--- Comment #5 from anish apa...@redhat.com 2011-08-30 06:45:08 EDT ---

Here's an unofficial review for F15,

+ rpmlint output

rpmlint output is  silent 
[makerpm@dhcp201-181 SOURCES]$ rpmlint luajit-2.0.0-0.2.beta8.fc16.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ License mentioned in the spec file are MIT  which looks okie

+ Sepc file is in American English 

+ Change log indicates the revised versions

Suggestions:-

1: According to packaging rules, please use naming conventions such as
lua-luajit

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-07-05 Thread bugzilla
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=718681

--- Comment #4 from Andrei Lapshin alaps...@gmx.com 2011-07-05 03:17:51 EDT 
---
Apply upstream hotfix #1

SPEC URL: http://dl.dropbox.com/u/11270386/luajit.spec
SRPM URL: http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.2.beta8.fc16.src.rpm

rpmlint -i SPECS/luajit.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

rpmlint -i SRPMS/luajit-2.0.0-0.2.beta8.fc16.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-07-04 Thread bugzilla
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=718681

Andrei Lapshin alaps...@gmx.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-07-04 Thread bugzilla
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=718681

--- Comment #1 from Damian L Brasher fed...@interlinux.org.uk 2011-07-04 
07:19:21 EDT ---
Informal review, initial check,

rpmlint -i luajit.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

rpmlint -i luajit-2.0.0-0.1.beta8.fc16.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Damian Brasher

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-07-04 Thread bugzilla
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=718681

--- Comment #2 from Andrei Lapshin alaps...@gmx.com 2011-07-04 13:28:31 EDT 
---
After build libluajit-5.1.so.2.0.0 was not executable and rpmbuild created
wrong cyclic dependencies. Fixed.

Spec URL: http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.1.beta8.fc16.src.rpm
SRPM URL: file:///home/alapshin/Dropbox/Public/luajit.spec


[alapshin@localhost rpmbuild]$ rpmlint -i SPECS/luajit.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

[alapshin@localhost rpmbuild]$ rpmlint -i
SRPMS/luajit-2.0.0-0.1.beta8.fc16.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 718681] Review Request: luajit - Just-In-Time Compiler for Lua

2011-07-04 Thread bugzilla
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=718681

--- Comment #3 from Andrei Lapshin alaps...@gmx.com 2011-07-04 14:12:48 EDT 
---
Sorry, SRPM URL:
http://dl.dropbox.com/u/11270386/luajit-2.0.0-0.1.beta8.fc16.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