[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-08-25 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=483025





--- Comment #10 from Milos Jakubicek xja...@fi.muni.cz  2009-08-25 21:17:17 
EDT ---
New Koji scratch build in case someone would review this:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1631677

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-06-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=483025





--- Comment #8 from Milos Jakubicek xja...@fi.muni.cz  2009-06-06 11:39:00 
EDT ---
Rebuilt, solved the ownership of audacious/General directory:

http://mjakubicek.fedorapeople.org/imms/imms.spec
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.5.rc8.fc10.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=1396981

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-06-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=483025





--- Comment #9 from Milos Jakubicek xja...@fi.muni.cz  2009-06-06 11:42:03 
EDT ---
wrong link by copypaste in the previous comment:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.6.rc8.fc10.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-03-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=483025





--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-03-13 16:20:13 EDT 
---
Unfortunately it fails to build for me:

+ execstack -c build/immsd build/immstool
RPM build errors:
/var/tmp/rpm-tmp.TIGCLm: line 41: execstack: command not found
error: Bad exit status from /var/tmp/rpm-tmp.TIGCLm (%install)

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-03-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=483025





--- Comment #7 from Milos Jakubicek xja...@fi.muni.cz  2009-03-13 20:38:48 
EDT ---
Ops, forgot to BR prelink after adding the execstack call, sorry.

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.5.rc8.fc10.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1240946

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-17 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=483025


Bug 483025 depends on bug 483020, which changed state.

Bug 483020 Summary: Review Request: torch - Torch is a simple machine-learning 
library
https://bugzilla.redhat.com/show_bug.cgi?id=483020

   What|Old Value   |New Value

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE



-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-15 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=483025





--- Comment #5 from Milos Jakubicek xja...@fi.muni.cz  2009-02-15 19:45:48 
EDT ---
OK, upstream is very responsive, hence there is a new (pre)release with all the
necessary licensing info:

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.4.rc8.fc10.src.rpm

Moreover, the GCC 4.3 patch has been merged upstream and the rpmlint issue with
executable stack has been also clarified, it is caused by a objdump call
(details in specfile) and has been simply fixed by removing the flag using
execstack.

I've also split the plugins into a xmms-imms and audacious-plugins-imms
subpackage.

So this is now definitely ready for a 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-14 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=483025


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Blocks|182235  |




--- Comment #4 from Tom spot Callaway tcall...@redhat.com  2009-02-14 
08:56:44 EDT ---
Yeah, this is sufficient for Fedora Legal. Lifting FE-Legal.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-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=483025





--- Comment #3 from Milos Jakubicek xja...@fi.muni.cz  2009-02-13 18:10:11 
EDT ---
Yep, I'm quoting the answer to my email:


Hey, Milos,

I fixed compilation with recent Audacious versions in 3.1.0-rc7.
[http://www.luminal.org/files/imms/imms-3.1.0-rc7.tar.bz2]

I am fairly sure all the code in question is GPL. I made sure of that
before including it.
I do not have the time to follow up with the authors of the code right
now, but I'm certainly willing to apply any patches to clarify
licenses.
Perhaps, Artur, the Debian IMMS maintainer (cc'ed) can help you there.
He already did some work on getting explicit releases from the
authors.

Have fun,
Michael. 


It seems that all the thins are GPL indeed...looking into debian(lenny) imms
packages, I found following:


+Author: Michael Grigoriev m...@luminal.org
+
+Copyright (C) 2003-2005 Michael Grigoriev
+
+With the following exceptions:
+
+md5.{h,c} (added ability to restrict maximum size to process)
+from GNU textutils
+levenshtein.{h,c} (stripped down)
+python-Levenshtein library
+by David Necas (Yeti) y...@physics.muni.cz
+regexx.{h,cc} (stripped down)
+Regexx - Regular Expressions C++ solution
+by Gustavo Niemeyer gust...@nn.com.br
+xidle.{h,cc}
+Borrows from xautolock 2.1
+by Michel Eyckmans (MCE) eyckm...@imec.be
+normal.h
+by by Agner Fog


I'll prepare patches and request adding a LICENSE file into the source tarball
and license information to all the source files (and merging the gcc43 patch of
course too). I'd also include the above information as %doc. Would it be ok
then?

Regarding the review: the audacious plugin is working now, updated link are
here:

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.3.rc7.fc10.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1125850

As there are now both xmms and audacious plugins available, I'm considering
splitting the package into a main one and two subpackages (-xmms, -audacious),
I'm just not sure whether this wouldn't hit the usability from the user point
of view too much.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-12 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=483025


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 CC||tcall...@redhat.com




--- Comment #2 from Tom spot Callaway tcall...@redhat.com  2009-02-12 
13:26:30 EDT ---
Any update on the licensing issue?

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-01-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=483025





--- Comment #1 from Milos Jakubicek xja...@fi.muni.cz  2009-01-30 08:26:23 
EDT ---
Torch is already built in rawhide, hence I made a scratch build here:

http://koji.fedoraproject.org/koji/taskinfo?taskID=1093697

I also added all the missing BR, had to add also automake + autoconf because of
the funny configure script provided in the source tarball:

cat configure
#!/bin/sh
rm $0
make
$0 $*

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.2.rc6.fc10.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-01-29 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=483025


Milos Jakubicek xja...@fi.muni.cz changed:

   What|Removed |Added

 Blocks||182235
 Depends on||483020




-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review