On Thu, Nov 18, 2010 at 14:03, Hans-Christoph Steiner <h...@at.or.at> wrote: > On Wed, 2010-11-17 at 16:50 -0300, Felipe Sateler wrote: >> On Sun, Nov 14, 2010 at 14:33, Hans-Christoph Steiner <h...@at.or.at> wrote: >> > >> > On Nov 13, 2010, at 8:37 PM, Felipe Sateler wrote: >> > >> >> On Thu, Nov 11, 2010 at 23:17, Hans-Christoph Steiner <h...@at.or.at> >> >> wrote: >> >>> >> >>> On Thu, 2010-11-11 at 09:19 +0100, Jonas Smedegaard wrote: >> >>>> >> >>>> On Thu, Nov 11, 2010 at 12:15:05AM -0500, Hans-Christoph Steiner wrote: >> >>>>> >> >>>>> pd-cyclone is a huge library of all sorts of objects that are clones of >> >>>>> objects in Max/MSP4.5, Pd's proprietary cousin. It also includes >> >>>>> utilities for importing Max files. >> >>>>> >> >>>>> pd-cyclone is a long-form dh package that is a wrapper to a very >> >>>>> complicated Make build system. pd-cxc is in >> >>>>> git.debian.org/pkg-multimedia. It is a library without depends that >> >>>>> are new packages. It builds on my i386, but it did not build on the >> >>>>> launchpad amd64 build machine, it died on this error, which has stumped >> >>>>> me: >> >>>>> >> >>>>> dpkg-genchanges -B -mUbuntu/amd64 Build Daemon <bui...@titanium.ppa> >> >>>>>> >> >>>>>> ../pd-cyclone_0.1~alpha55-1~maverick_amd64.changes >> >>>>> >> >>>>> dpkg-genchanges: arch-specific upload - not including arch-independent >> >>>>> packages >> >>>>> dpkg-genchanges: error: cannot read files list file: No such file or >> >>>>> directory >> >>>> >> >>>> It looks odd to me that you use no debhelper tools in the binary-arch >> >>>> target - only in the binary-indep one. >> >>> >> >>> Thanks, that was the key. This now builds on i386 and amd64 and is >> >>> ready to go. >> >> >> >> The git repository is not fixed. >> > >> > I am not sure I understand. What I see on git.debian.org is the same as >> > what >> > I have on my machine, and I used git-buildpackage to make the source >> > package >> > to submit to Launchpad: >> > >> > http://launchpad.net/~eighthave/+archive/libdirs/+sourcepub/1362005/+listing-archive-extra >> >> Oh it was fixed, my bad. But there is a large commit that changes many >> things at once (d634143), and the change was hidden there. >> >> > >> >> Also, why do you use long-form debhelper? >> > >> > >> > Because I didn't see a way to do it with short-form. This library has a >> > very >> > elaborate build system that I've tried for years to figure out. Instead >> > I've found the best way is to wrap it into something more understandable, >> > and long-form seemed the easiest way to do that. >> >> Indeed, the makefile is quite convoluted. It also can't handle it when >> the user passes CFLAGS and other variables. Also, it sets -j4 >> unconditionally. > >> On further inspection, the package seems to bundle the pddp source. >> Are riddle, sickle and ViCious more code copies? We should not use >> code copies. We need to avoid using them. > > This is how cyclone is released. The code in 'pddp', 'riddle', and > 'ViCious' are unused. The 'pddp' code shared some code with pd-pddp, > but not all. The code in 'sickle' is fully part of pd-cyclone and used.
OK. Are you willing to look into a switch to cdbs (using long-form debhelper does not make much sense, and I personally prefer cdbs over dh)? I have pushed a cdbs branch to the git repository for you to look at. If you are willing to give cdbs a try, we can merge into the master branch and I can continue the review and upload. -- Saludos, Felipe Sateler _______________________________________________ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers