[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-10-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=454215


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-10-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=454215





--- Comment #25 from Fedora Update System [EMAIL PROTECTED]  2008-10-03 
18:27:50 EDT ---
stk-4.3.1-6.fc8 has been pushed to the Fedora 8 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.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-10-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=454215





--- Comment #26 from Fedora Update System [EMAIL PROTECTED]  2008-10-03 
18:35:40 EDT ---
stk-4.3.1-6.fc9 has been pushed to the Fedora 9 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.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-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=454215


Fedora Update System [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #23 from Fedora Update System [EMAIL PROTECTED]  2008-09-24 
20:01:40 EDT ---
stk-4.3.1-6.fc8 has been pushed to the Fedora 8 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 stk'.  You can provide feedback
for this update here:
http://admin.fedoraproject.org/updates/F8/FEDORA-2008-8185

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-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=454215





--- Comment #24 from Fedora Update System [EMAIL PROTECTED]  2008-09-24 
20:23:29 EDT ---
stk-4.3.1-6.fc9 has been pushed to the Fedora 9 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 stk'.  You can provide feedback
for this update here:
http://admin.fedoraproject.org/updates/F9/FEDORA-2008-8333

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-19 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=454215





--- Comment #21 from Fedora Update System [EMAIL PROTECTED]  2008-09-19 
06:39:53 EDT ---
stk-4.3.1-6.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/stk-4.3.1-6.fc8

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-19 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=454215





--- Comment #22 from Fedora Update System [EMAIL PROTECTED]  2008-09-19 
06:41:07 EDT ---
stk-4.3.1-6.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/stk-4.3.1-6.fc9

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-16 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=454215





--- Comment #18 from Nicolas Chauvet (kwizart) [EMAIL PROTECTED]  2008-09-16 
10:46:59 EDT ---
Package built fine in Rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=828097


This package (stk) is APPROVED by me

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-16 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=454215





--- Comment #19 from Thomas Moschny [EMAIL PROTECTED]  2008-09-16 14:26:55 
EDT ---
Thanks for the review!


New Package CVS Request
===
Package Name: stk
Short Description: Synthesis ToolKit in C++
Owners: thm
Branches: F-8 F-9
InitialCC: none
Cvsextras Commits: yes

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-16 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=454215


Huzaifa S. Sidhpurwala [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #20 from Huzaifa S. Sidhpurwala [EMAIL PROTECTED]  2008-09-16 
23:15:55 EDT ---
cvs done

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-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=454215





--- Comment #14 from Nicolas Chauvet (kwizart) [EMAIL PROTECTED]  2008-09-09 
12:58:35 EDT ---
Can you split %{_libdir}/libstk.a into another package (-static)
if the static version is really needed. (I expect not...)

Then I think we are good...

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-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=454215





--- Comment #15 from Thomas Moschny [EMAIL PROTECTED]  2008-09-09 14:55:54 
EDT ---
Spec URL: http://thm.fedorapeople.org/stk.spec
SRPM URL: http://thm.fedorapeople.org/stk-4.3.1-5.fc9.src.rpm

%changelog
* Tue Sep  9 2008 Thomas Moschny ... - 4.3.1-5
- Don't ship the static library.

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-09-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=454215





--- Comment #13 from Thomas Moschny [EMAIL PROTECTED]  2008-09-08 03:08:53 
EDT ---
Thanks, spot. So, for now, I removed them all.

Spec URL: http://thm.fedorapeople.org/stk.spec
SRPM URL: http://thm.fedorapeople.org/stk-4.3.1-4.fc9.src.rpm

%changelog
* Sun Sep  7 2008 Thomas Moschny ... - 4.3.1-4
- Remove all .mid and .ski files from the tarball.
- Add README.fedora.

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-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=454215


Tom spot Callaway [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #10 from Tom spot Callaway [EMAIL PROTECTED]  2008-09-02 
12:29:46 EDT ---
What is the legal concern here? Do you just want me to do a general audit on
the package?

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-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=454215





--- Comment #11 from Thomas Moschny [EMAIL PROTECTED]  2008-09-02 14:27:54 
EDT ---
(In reply to comment #10)
 What is the legal concern here? Do you just want me to do a general audit on
 the package?

In short: yes.

Long answer: Even after talking to upstream, the legal status of some files
remains unclear to me, especially that of the .mdi and .ski files.

Quoting Gary Scavone, one of the authors:

... I think there should be no issue with the .ski files because most were
originally created by Perry Cook himself.  It is possible a few were created by
Craig Sapp's MIDI to SKINI converter from MIDI files (though only Bach pieces).
 As for the MIDI files, I downloaded a few over time, like the bach fugue.  I
think the copyright notice you see there (David Huron) is for the software that
was used to do the MIDI to SKINI conversion.  Bach's music is clearly no longer
copyright protected.  So, if there's any issue, it _might_ be the tango, but I
don't know if that has an author.  Worst case ... just delete the tango file,
since the other stuff is Bach and the rights are clear. ...


While the music (composition) itself might not be copyright protected
anymore, a certain performance might be, I think.

There are three midi files:

- bwv772.mid has a notice that says the harpsichord was played by
John Sankey. Here's his copyright notice: http://www.sankey.ws/copyright.html.

- tango.mid has Copyright 1996 R Finley, might be that one:
http://www.classicalarchives.com/info/tango.txt.

- jesu.mid has Copyright (C) 1992 MTA All rights reserved. I don't know
who/what MTA is.

Not sure what to do about these three files, and I think some of the .ski files
might be problematic as well.

Easiest solution would be to simply remove all of them.

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-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=454215





--- Comment #12 from Tom spot Callaway [EMAIL PROTECTED]  2008-09-02 
16:04:21 EDT ---
Agreed. Remove them all.

If and only if you can:
 * clearly determine the copyright holder
 * explicitly confirm from the copyright holder that we have permission to use 
   the content under an acceptable license.

Then, you can include those files on a case-by-case basis.

It might not be a bad idea to look for some Creative Commons licensed mid or
ski files as possible replacements, or to ask on some of the Fedora lists if
any musicians want to make some for you. Only thing you need to avoid are any
of the files which are marked NC or No Commercial.

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-08-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=454215


Thomas Moschny [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||182235




-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-08-10 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=454215





--- Comment #9 from Thomas Moschny [EMAIL PROTECTED]  2008-08-10 08:38:59 EDT 
---
Current status of this package:

Spec URL: http://thm.fedorapeople.org/stk.spec
SRPM URL: http://thm.fedorapeople.org/stk-4.3.1-3.fc9.src.rpm

%changelog
* Thu Jul 31 2008 Thomas Moschny ... - 4.3.1-3
- Remove src/include/dsound.h, and src/include/*asio* files from the
  tarball, for legal reasons. Only used on windows anyway.
- Remove src/include/soundcard.h (explicitly forbids modification) and
  disable OSS support.
- Build and pack Md2Skini.
- Build and pack the examples.

-- 
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 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-21 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-21 02:51 EST ---
(In reply to comment #7)
 There is a list of undefined-non-weak-symbol that are usually the sign of a
 missing LIB_ADD Flagh at linkig time. -ljack -lpthread and others
 Theses symbols will be resolved at runtime if not specified are linking time.
 And may silently break if they are not found.

Do you really see that with my latest version, 4.3.1-2 (see comment #4)?

 stk.x86_64: W: incoherent-version-in-changelog 4.3.1-1 4.3.1-1.fc8.kwizart

Hmm, seems you checked on old 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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-20 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 06:57 EST ---
for src/include/dsound.h I expect it shouldn't be ship in this release. Instead
win32 users will need to download this file separely.
for src/include/soundcard.h, This file explicitely prevent modification. As such
we cannot allow this. But there is another soundcard.h (licensed as GPLv2 only)
that can be downloaded from http://www.4front-tech.com/developer/sources/stable/
There is others license available, but I think linux support will need the GPLv2
version. 
So i think there is two solutions: 
- repack the sources with soundcard.h Gplv2 version (and remove
src/include/dsound.h will be needed anyhow). 
- Or disable OSS support with the filed removed.
Fedora doesn't use OSS anyway (only emulated from alsa), so if needed, it could
be possible to have a replacement version (in _libdir/stk-oss ). My guess is
that ABI compatibility will be preserved in the dependent applications (but
maybe i'm wrong).
For the ski,mid file, it would be fine to have the respective authors to be
aware of providing a free license (Creative commons?) when they are provided
within FOSS. Not all files have an Author but there is at least:
///YEM: Rights to all derivative electronic formats reserved.
in projects/examples/scores/bachfugue.ski
If unsure, they can still be removed... 

Anyway it would be nice to ask upstream for advices on each cases.


-- 
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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-20 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 17:17 EST ---
There is a list of undefined-non-weak-symbol that are usually the sign of a
missing LIB_ADD Flagh at linkig time. -ljack -lpthread and others
Theses symbols will be resolved at runtime if not specified are linking time.
And may silently break if they are not found.

stk.x86_64: W: incoherent-version-in-changelog 4.3.1-1 4.3.1-1.fc8.kwizart
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_set_channels
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_set_period_size_near
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
snd_pcm_readi
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
jack_on_shutdown
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
jack_port_name
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_set_rate_near
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_info_get_name
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_free_event
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_subscribe_free
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 jack_connect
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_info_get_client
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_event_output
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_state_name
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_info_set_port
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
jack_port_by_name
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_client_info_sizeof
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_midi_event_init
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_set_access
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
jack_set_process_callback
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 snd_strerror
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_client_id
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_format_cpu_endian
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_info_set_device
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_event_input
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
snd_pcm_close
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
jack_deactivate
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
jack_client_new
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
jack_port_register
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_create_simple_port
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
jack_set_xrun_callback
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_info_set_subdevice
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_subscribe_set_time_real
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_set_periods_near
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
snd_pcm_writen
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_drain_output
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_card_get_name
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
jack_port_get_buffer
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_port_info_set_timestamping
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
snd_pcm_state
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_query_next_client
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_queue_tempo_set_tempo
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 
jack_get_ports
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_pcm_hw_params_get_channels_min
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_seq_unsubscribe_port
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1 snd_seq_open
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_midi_event_resize_buffer
stk.x86_64: W: undefined-non-weak-symbol /usr/lib64/libstk.so.4.3.1
snd_midi_event_free
stk.x86_64: W: undefined-non-weak-symbol 

[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-16 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:11 EST ---
Spec URL: http://thm.fedorapeople.org/stk.spec
SRPM URL: http://thm.fedorapeople.org/stk-4.3.1-2.fc9.src.rpm

%changelog
* Tue Jul 15 2008 Thomas Moschny .. - 4.3.1-2
- Update sharedlib patch, fixes alsa problem.
- Fix path for include files in -devel.
- Change path for docs in -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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-16 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:15 EST ---
Here's the list of files with (at least to me) unclear legal status

 - src/include/dsound.h
 - src/include/soundcard.h
 - projects/demo/scores/*.ski
 - projects/examples/scores/*.ski
 - projects/examples/midifiles/*.mid
 - doc/html/tutorial/*.ski

It would be nice if someone could comment or help me sorting this out.

-- 
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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-08 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-08 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-08 12:33 EST ---
That' very strange, they use an autotools configure script but the Makefile.in
isn't automake compliant. Is there any reasons not to move to an plain
Makefile.am file (using automake and probably libtool also)?

Do you know if there is problem while realtime is enabled and if it will lead to
ABI changes ?

-- 
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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-08 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-08 14:27 EST ---
(In reply to comment #1)
 That' very strange, they use an autotools configure script but the Makefile.in
 isn't automake compliant.

I think it is perfectly alright to use autoconf but not automake. 

 Is there any reasons not to move to an plain Makefile.am file (using automake
 and probably libtool also)?

That'd be an upstream decision, and it would probably alleviate building the
shared lib for us. But honestly, my sharedlib patch isn't that complicated, and
as far as I see, it's working. So I'm not going to waste my time struggling with
autotools, unless I can clearly see what gain that could have. We don't care
whether it's portable or something, as long as it works on Fedora.

Other way round, do you see anything not working properly the way it is now?

 Do you know if there is problem while realtime is enabled and if it will lead
 to ABI changes ?

Can you rephrase, please? I don't understand that question.

-- 
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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-08 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||443771




-- 
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, or are watching someone who is.

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


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-08 Thread bugzilla
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: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|443771  |
OtherBugsDependingO||443771
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-08 14:39 EST ---
Ops. Wrong order of bug dependency, sorry.

-- 
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, or are watching someone who is.

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