Spec URL: http://music.nuim.ie/vlazzarini/tmp/SndObj.spec SRPM URL: http://music.nuim.ie/vlazzarini/tmp/SndObj-2.6.6-0.fc7.src.rpm Description: This the package of the SndObj library, a general-purpose audio programming library. This is my second package and I would like it to be reviewed as part of the process of my getting sponsored.
For what branches (i.e. Fedora-[10-8] or olpc) do you want to import this package?
If possible, both; otherwise just Fedora
Well, your srpm does not build on either dist-f10 or dist-f9-updates-candidate: http://koji.fedoraproject.org/koji/taskinfo?taskID=633523 http://koji.fedoraproject.org/koji/taskinfo?taskID=633567 Unfortunately I am not scons expert and perhaps I don't know what is going on.
It's OK, I can see what the problem is, there is a Command target that scons does not like. I'll fix this tomorrow. Thanks.
I think I have the scons problem fixed now. Here are the new packages Spec URL: http://music.nuim.ie/vlazzarini/tmp/SndObj.spec SRPM URL: http://music.nuim.ie/vlazzarini/tmp/SndObj-2.6.6-1.fc7.src.rpm Thanks
This time build fails with another reason (perhaps gcc43 header files inclusion change) http://koji.fedoraproject.org/koji/taskinfo?taskID=635118
OK. Do you know where memset() is in gcc43? This seems easy to fix. Thanks
(In reply to comment #7) > OK. Do you know where memset() is in gcc43? man memset ... #include <string.h> => #include <cstring>
Created attachment 307156 [details] patch to address cstring issues
Removing NEEDSPONSOR, I am sponsoring
Ralf, thank you for the patch. Well, even after applying the patch in comment 9, the build still fails (due to another reason) http://koji.fedoraproject.org/koji/taskinfo?taskID=650441
Any response, Victor? It's been a month since your last comment.
Sorry, but I have not had time to work on this. I have fixes for everything and now I can test it on Koji. But it will be mid-July before I can get around to doing it. I am taking this off the urgent priority.
OK, just clear the status whiteboard when you're ready and this will reappear in the review queue.
What is the status of this package?
Not ready for re-review yet. I have to fix some of the scons build issues before I set it for approval.
Any updates here?
I have closed this bug for the moment. The package needs some extra work that will take a little longer than I predicted. I will open a new ticket when I have it ready for review again.
Okay, feel free to open a new review request when it is ready.