[Bug 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-23 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=567028

--- Comment #20 from Fedora Update System upda...@fedoraproject.org 
2010-09-23 08:32:56 EDT ---
lv2-ll-plugins-0.2.8-4.fc14 has been pushed to the Fedora 14 stable repository.
 If problems still persist, please make note of it in this bug report.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-23 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=567028

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|lv2-ll-plugins-0.2.8-4.fc13 |lv2-ll-plugins-0.2.8-4.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-22 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=567028

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2010-09-23 00:55:05 EDT ---
lv2-ll-plugins-0.2.8-4.fc12 has been pushed to the Fedora 12 stable repository.
 If problems still persist, please make note of it in this bug report.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-22 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=567028

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||lv2-ll-plugins-0.2.8-4.fc12
 Resolution||ERRATA
Last Closed||2010-09-23 00:55:11

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-22 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=567028

--- Comment #19 from Fedora Update System upda...@fedoraproject.org 
2010-09-23 00:59:02 EDT ---
lv2-ll-plugins-0.2.8-4.fc13 has been pushed to the Fedora 13 stable repository.
 If problems still persist, please make note of it in this bug report.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-22 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=567028

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|lv2-ll-plugins-0.2.8-4.fc12 |lv2-ll-plugins-0.2.8-4.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-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=567028

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-09-06 01:15:24 EDT ---
lv2-ll-plugins-0.2.8-4.fc14 has been pushed to the Fedora 14 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update lv2-ll-plugins'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/lv2-ll-plugins-0.2.8-4.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-03 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=567028

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-03 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=567028

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2010-09-03 23:53:48 EDT ---
lv2-ll-plugins-0.2.8-4.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/lv2-ll-plugins-0.2.8-4.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-03 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=567028

--- Comment #16 from Fedora Update System upda...@fedoraproject.org 
2010-09-03 23:54:14 EDT ---
lv2-ll-plugins-0.2.8-4.fc12 has been submitted as an update for Fedora 12.
https://admin.fedoraproject.org/updates/lv2-ll-plugins-0.2.8-4.fc12

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-03 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=567028

--- Comment #15 from Fedora Update System upda...@fedoraproject.org 
2010-09-03 23:54:08 EDT ---
lv2-ll-plugins-0.2.8-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/lv2-ll-plugins-0.2.8-4.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-02 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=567028

Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-09-02 
02:47:04 EDT ---
Thanks for the review!

New Package SCM Request
===
Package Name: lv2-ll-plugins
Short Description: Collection of LV2 plugins
Owners: oget
Branches: F-12 F-13 F-14
InitialCC:

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-09-02 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=567028

--- Comment #13 from Kevin Fenzi ke...@tummy.com 2010-09-02 17:28:53 EDT ---
Git done (by process-git-requests).

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-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=567028

David Cornette r...@davidcornette.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #11 from David Cornette r...@davidcornette.com 2010-08-25 
03:10:10 EDT ---
Looks good.

APPROVED

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-08-22 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=567028

--- Comment #9 from David Cornette r...@davidcornette.com 2010-08-22 16:31:10 
EDT ---
It sounds like a good idea to not package elven.  If it is a separate project
now, that makes things a little tricky going forward.  You can imagine a user
upgrade a plugin, and suddenly a program they have been using is gone.  Or this
plugin package would have to require an elven package, even though many users
have no need for it.

SO if you want to remove elven, and add the lv2core require, this should be
good to go.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-08-22 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=567028

--- Comment #10 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-08-22 
19:59:24 EDT ---
Alright

SPEC: http://oget.fedorapeople.org/review/lv2-ll-plugins.spec
SRPM: http://oget.fedorapeople.org/review/lv2-ll-plugins-0.2.8-4.fc13.src.rpm

changelog 0.2.8-4
- Add R: lv2core
- Don't build and package elven as it is now a separate project
- Remove BR: jack-audio-connection-kit-devel boost-devel

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-08-08 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=567028

--- Comment #8 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-08-08 
22:35:28 EDT ---
Hello, sorry I was too busy in the past week with the python-2.7 rebuilds for
Fedora 14.

(In reply to comment #7)
 There are a few issues to address.
 
 Here is the output of rpmlint:
 lv2-ll-plugins.src: W: spelling-error %description -l en_US arpeggiator -
 arpeggiate, arpeggio, appoggiatura
 lv2-ll-plugins.src:39: W: configure-without-libdir-spec
 lv2-ll-plugins.x86_64: W: spelling-error %description -l en_US arpeggiator -
 arpeggiate, arpeggio, appoggiatura
 lv2-ll-plugins.x86_64: W: no-manual-page-for-binary elven
 /home/makerpm/rpmbuild/SPECS/lv2-ll-plugins.spec:39: W:
 configure-without-libdir-spec
 
 Spelling is fine.  I suppose we could file a bug against the words package if
 we wanted to be helpful, but that isn't necessary.
 
 You should email upstream about a man page.
 

I talked to upstream about elven. Apparently, elven became a separate project:
   http://gitorious.org/elven/elven

The next release of the ll plugins will not include elven. I should probably
drop elven from our RPM. What do you think?

 I understand that configure isn't generated by autotools, but I am not sure
 you couldn't specify libdir instead of specifying lv2plugindir.  It looks like
 lv2plugindir will be set to $(libdir)/lv2 by default, so why not just set
 libdir?  Am I missing something?
 

Well, we can do it either way. It really doesn't matter. This is more of a
convention. I don't think this should be a blocker.

 Since the elven package is capable of displaying some plugins with guis, 
 should
 there be .desktop files for it to start the included gui plugins?
 

See above about dropping elven. But I don't think we should include a .desktop
file. If we do, we need to supply a .desktop file for every lv2 plugin we ship.
And this means that the elven package needs to be updated whenever some
lv2-X-plugins package gets included in Fedora. This sounds impractical.

 This package needs to Require: lv2core for directory ownership.

Ah good catch. I will add this.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-08-01 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=567028

David Cornette r...@davidcornette.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?

--- Comment #7 from David Cornette r...@davidcornette.com 2010-08-01 03:30:47 
EDT ---
There are a few issues to address.

Here is the output of rpmlint:
lv2-ll-plugins.src: W: spelling-error %description -l en_US arpeggiator -
arpeggiate, arpeggio, appoggiatura
lv2-ll-plugins.src:39: W: configure-without-libdir-spec
lv2-ll-plugins.x86_64: W: spelling-error %description -l en_US arpeggiator -
arpeggiate, arpeggio, appoggiatura
lv2-ll-plugins.x86_64: W: no-manual-page-for-binary elven
/home/makerpm/rpmbuild/SPECS/lv2-ll-plugins.spec:39: W:
configure-without-libdir-spec

Spelling is fine.  I suppose we could file a bug against the words package if
we wanted to be helpful, but that isn't necessary.

You should email upstream about a man page.

I understand that configure isn't generated by autotools, but I am not sure
you couldn't specify libdir instead of specifying lv2plugindir.  It looks like
lv2plugindir will be set to $(libdir)/lv2 by default, so why not just set
libdir?  Am I missing something?

Since the elven package is capable of displaying some plugins with guis, should
there be .desktop files for it to start the included gui plugins?

This package needs to Require: lv2core for directory ownership.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-07-28 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=567028

--- Comment #6 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-07-28 
02:41:28 EDT ---
Hello, slightly off-topic but,
I just wanted to warn you that I rebuilt lv2-c++-tools against the new boost on
rawhide. Since lv2-c++-tools contains a static library you need to rebuild
lv2-EQ10Q-plugins against it. This is for rawhide only. Thanks.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-07-26 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=567028

--- Comment #5 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-07-27 
00:57:35 EDT ---
Apparently, I never built this on mock, because when I prepared it for review,
lv2-c++-tools was not in the repos. I added the missing BRs.

The DSO error is something that came with F-13. From F-13 on, Fedora does not
allow indirect linking to libraries. So I added the missing link.

Thanks for the warnings.

SPEC: http://oget.fedorapeople.org/review/lv2-ll-plugins.spec
SRPM: http://oget.fedorapeople.org/review/lv2-ll-plugins-0.2.8-3.fc13.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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-07-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=567028

Bug 567028 depends on bug 537363, which changed state.

Bug 537363 Summary: Review Request: lv2-c++-tools - Support library for writing 
LV2 plugins in C++
https://bugzilla.redhat.com/show_bug.cgi?id=537363

   What|Old Value   |New Value

 Status|ON_QA   |CLOSED
 Resolution||ERRATA

--- Comment #3 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2010-07-25 
15:42:41 EDT ---
Hello, are you still reviewing this? If not, let's set the Assigned To to
null. Thanks.

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-07-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=567028

--- Comment #4 from David Cornette r...@davidcornette.com 2010-07-26 00:52:04 
EDT ---
I have started to take a look at this package, and I am unable to get it to
build in mock.  It looks like it may be missing some BuildRequires.  It looks
like you need to add lv2core-devel and boost-devel.

I tried building it outside of mock, where I already have those two packages
installed, and got this error, which if I am not mistaken is a common one on
F13:

/usr/bin/ld: programs/elven/lv2host.o: undefined reference to symbol
'dlclose@@GLIBC_2.2.5'
/usr/bin/ld: note: 'dlclose@@GLIBC_2.2.5' is defined in DSO /lib64/libdl.so.2
so try adding it to the linker command line
/lib64/libdl.so.2: could not read symbols: Invalid operation

-- 
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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-05-18 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=567028

David Cornette r...@davidcornette.com changed:

   What|Removed |Added

 CC||r...@davidcornette.com
 AssignedTo|nob...@fedoraproject.org|r...@davidcornette.com

--- Comment #2 from David Cornette r...@davidcornette.com 2010-05-18 23:24:26 
EDT ---
I'll review this once its dependency is 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 567028] Review Request: lv2-ll-plugins - Collection of LV2 plugins

2010-02-20 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=567028

Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

 Depends on||537363

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