looking for feedback on operator-name-cbs-widget

2011-10-25 Thread Jonathan Wilson
I uploaded the first pass of my code here: http://www.cncmods.net/files/operator-name-cbs-widget-0.1.zip It works pretty good, it applies the patch and it displays the right output (except for the operator name which I am still working on) but I want further feedback on what I should be doing d

Re: Looking for some help with autoconf/automake/dpkg (and general feedback on my code)

2011-10-25 Thread Neal H. Walfield
At Tue, 25 Oct 2011 14:07:13 +0800, Jonathan Wilson wrote: > I suspect also I need more in the postinst file in terms of verifying the > correct binary and things, any help would be appreciated. You should never modify a binary in place. Instead, copy the binary, modify it and then atomically re

Re: Looking for some help with autoconf/automake/dpkg (and general feedback on my code)

2011-10-25 Thread Daniil Ivanov
I second proposition to use CBDS instead of debhelper directly. Thanks, Daniil On Tue, Oct 25, 2011 at 11:13 AM, Jonathan Wilson wrote: > Daniil Ivanov wrote: >> At least debian/install is missing in debianization of a package >> http://www.debian.org/doc/manuals/maint-guide/dother.en.html#insta

Re: Looking for some help with autoconf/automake/dpkg (and general feedback on my code)

2011-10-25 Thread Jonathan Wilson
Daniil Ivanov wrote: > At least debian/install is missing in debianization of a package > http://www.debian.org/doc/manuals/maint-guide/dother.en.html#install Thanks, that's a start. Still cant get it to even produce a cbspatch binary though. ___ maem

Re: Looking for some help with autoconf/automake/dpkg (and general feedback on my code)

2011-10-25 Thread Neal H. Walfield
At Tue, 25 Oct 2011 14:07:13 +0800, Jonathan Wilson wrote: > > I have uploaded my code and files to > http://www.cncmods.net/files/operator-name-cbs-widget-0.1.zip > > I am looking for general feedback on this code and packaging but > specifically I am having problems getting it to create, inclu

Re: Looking for some help with autoconf/automake/dpkg (and general feedback on my code)

2011-10-25 Thread Daniil Ivanov
At least debian/install is missing in debianization of a package http://www.debian.org/doc/manuals/maint-guide/dother.en.html#install Thanks, Daniil On Tue, Oct 25, 2011 at 9:07 AM, Jonathan Wilson wrote: > I have uploaded my code and files to > http://www.cncmods.net/files/operator-name-cbs-wid