Carsten Schlote wrote:
> From: Carsten Schlote <c.schl...@konzeptpark.de>
> 
> Added simple tool to monitor modem status lines on serial ports. Either 
> display
> line status with an ncurses interface or return a decimal or hex value.
> 
> Useful to check operation of serial ports with an EOL test.
> 
> Signed-off-by: Carsten Schlote <c.schl...@konzeptpark.de>
> ---
>  .../statserial-1.1.orig/crosscompile-fixes.patch   |   42 +++++++++++
>  patches/statserial-1.1.orig/series                 |    2 +
>  .../statserial-1.1.orig/statserial_1.1-22.diff.gz  |  Bin 0 -> 3135 bytes
>  rules/statserial.in                                |    7 ++
>  rules/statserial.make                              |   74 
> ++++++++++++++++++++
>  5 files changed, 125 insertions(+), 0 deletions(-)
>  create mode 100644 patches/statserial-1.1.orig/crosscompile-fixes.patch
>  create mode 100644 patches/statserial-1.1.orig/series
>  create mode 100644 patches/statserial-1.1.orig/statserial_1.1-22.diff.gz
>  create mode 100644 rules/statserial.in
>  create mode 100644 rules/statserial.make
> 
> diff --git a/patches/statserial-1.1.orig/crosscompile-fixes.patch 
> b/patches/statserial-1.1.orig/crosscompile-fixes.patch
> new file mode 100644
> index 0000000..e9481cc
> --- /dev/null
> +++ b/patches/statserial-1.1.orig/crosscompile-fixes.patch
> @@ -0,0 +1,42 @@
> +[crosscompile fixes] Fixed up Makefile to support cross-compilation
> +
> +- Allow for CC and LD to be defined externally.
> +- Allow to define CPPFLAGS and LDFLAGS externally
> +- Added DESTDIR prefix for install stage
> +
> +Signed-off-by: Carsten Schlote <c.schl...@konzeptpark.de>
> +
> +Index: statserial-1.1.orig/Makefile
> +===================================================================
> +--- statserial-1.1.orig.orig/Makefile        2010-04-16 16:44:08.000000000 
> +0200
> ++++ statserial-1.1.orig/Makefile     2010-04-16 17:17:11.000000000 +0200
> +@@ -1,13 +1,14 @@
> +-CC  = gcc
> +-LD  = gcc
> ++
> ++CC  ?= gcc
> ++LD  ?= gcc
> + 
> + # for debug
> + #CFLAGS     = -Wall -g
> + #LDFLAGS = -N
> + 
> + # for production code
> +-CFLAGS      = -Wall -O3 -fomit-frame-pointer
> +-LDFLAGS = -s
> ++CFLAGS      = -Wall -O3 -fomit-frame-pointer ${CPPFLAGS}

please use "CFLAGS +=" so that they can be extended from outside.

> ++LDFLAGS ?= -s

..same goes for LDFLAGS. Remove the "-s", make install should install
unstripped programs/libs (this may go into a seperate patch, though)

> + 
> + statserial: statserial.o
> +     $(LD) $(LDFLAGS) -o statserial statserial.o -lcurses
> +@@ -16,8 +17,8 @@
> +     $(CC) $(CFLAGS) -c statserial.c
> + 
> + install: statserial
> +-    install -m 555 statserial /usr/local/bin/statserial
> +-    install -m 444 statserial.1 /usr/local/man/man1/statserial.1
> ++    install -m 555 statserial $(DESTDIR)/usr/bin/statserial
> ++    install -m 444 statserial.1 $(DESTDIR)/usr/man/man1/statserial.1

Maybe it's better to introduce something like "PREFIX = /usr/local" that
can be overwritten from outside.

> + 
> + clean:
> +     $(RM) statserial statserial.o core *~
> diff --git a/patches/statserial-1.1.orig/series 
> b/patches/statserial-1.1.orig/series
> new file mode 100644
> index 0000000..141e03f
> --- /dev/null
> +++ b/patches/statserial-1.1.orig/series
> @@ -0,0 +1,2 @@
> +statserial_1.1-22.diff.gz
> +crosscompile-fixes.patch
> diff --git a/patches/statserial-1.1.orig/statserial_1.1-22.diff.gz 
> b/patches/statserial-1.1.orig/statserial_1.1-22.diff.gz
> new file mode 100644
> index 
> 0000000000000000000000000000000000000000..6886c6a43363a00f06eb664dce15f9cb741823e3
> GIT binary patch
> literal 3135
> zcmV-F48ZdriwFQWf-6G;1EpE}Q`<-q{u%!jeP-`~7|Cw~CQ0#5vdbeClik|7...@hc
> z*is>DTp9`R?%n@>U(bw;Ot697JPaE3^z?lFZaa=cD)TZW(#vuqrrui78...@yp!3z
> z)oPE37T+Hnkke^x?@&#D9voDtLiIN^j^)sc>DB|?Fcym5)GOqNUP^(ujePO%R=4|~
> zeOmelED~=bNRI3V$|T9DNSKl1%S2^{-J3v+ygbfM4AncT*0G`P=U%hAZx~sMf>AW0
> zL?(jNRQS;-5&_ACd^wqTNkDOgH#nm9{1I6;8~m*^C`vL)zR778;q5zWtj0r$jQKhV
> z*&UIhT!D4LDtI-kU>U3XSF^f*f)yj...@ogvscca5kx;QiEmyyR-5?nw+WxOTl;J|
> znx$6rRHm8MHPI-57`(V?(A}o1Z(mNXF3*3(rrN6a^k4jXd3AYoes%nleK@|p=6yH{
> zVsG-^caU<6{;crlx^k...@pz&Q$RA>RNGX2u>C&yUb2xHVXfAG|EJohZrdlE;Vkj6
> z3-3mZqF6lTUGsMg-VQ^zGxw`?8p)KVsSI*oW9`d8ROtBZ{P4qI4cGX^i(_)g...@jn
> zB+8so3LkQ&67q|*;+&soN}%N^axN>i@&x...@fy53sj&1Hqwe%L%e*&_bbTeirNC2}fL
> zlN0+nRIHW)p...@--=6jnygh%n)s...@~1+~2ai2)JkvF%Gb}BoHaxg^?dp7QzG>LYjCp
> z8VX895UDJUhIu9e7yGY2(%|~=dT{#X)8RQasoLD36XA==5KNkTdz~s(&!Z&2$Iq{j
> zgn}{o...@qkgnmk#c9pu!$rs2ghqq=pp8l9fg1gumapl;`...@m#lc?^eny!t%&G+2sJ
> zAcm2b)O|Uf;W5mf!s$C2#;UR1fWK|G8VpwLn$f_&EEKpSlhcRGuXTMrn39-!{*5<=
> zzut;eu}vZ3mf)|{1`n6K^Az2sQI^5vhco(7j7D^wdTd<u...@ar-o?q6r~@i...@84m
> zH68OjCd#J)eg?ex43sx?<mHpG2cX`LWgxSIJc*pC2VD+jnz(^...@4*~2qte{$mx`ue;
> zj0G;0GP4PM{y0pbWxgNo7HXb8FC7?=0P6TXGdp9;3B6>Dr1S|U5(sTfh6>6...@mf
> zS70IPf%HqpSU3A(Id%<{...@u;LyPQ@|m?iFna%*MRAnP4DoTHE80BX(#E!k<L1*i
> zZkgQ%iByi^*sRr23VD?rWp|KvpJp=Wu(V+D6lDYr)Nw~xV>FsEAFLUdVY9$N6UBPx
> z2N(x^5Q#|n68SKWeBkMeL<!QsV?~8ph0KR{oUsf8D+8TLF!tCh_en%rL4{?tiZd+3
> zZIUT7+4M4&Jhd5Q3*yeK!Pu?Z3a*mx4Oq=oCMJl?%oh1|8jFlCbe}zOpGxh~g&3+K
> zs~;jG?a}Wg&2zGri4z-ARsZn$e1pY!&Rq)&S>>*Cnv%Qj?P_-|+0rJqIIY$eG=)_Z
> zQTvpqW0Cl?cd@)p_v5ov1`%ig?qckrppttsyfit%hpm...@}1wk!m_<XQ(ZbVsR^C
> z>KLL^1XCz>9!yLO1+P(b&#_DUmW%^uq(+Vx1gTKUi4teN!7(`v+<?)~QG$$kk;kEo
> zV<`XaZ9FNvBEfE=ZEE)5F;i+a8eM8N`>jsDf$Y`jHX6+6J2aYq6^*XD>oyF{S#%HA
> zM?9Y*oH(JUoS_#ucOhU=j!3d}Mw^3B=5c@;4po(e(QqZ_`J&jRw#uxg5^)DoIZE$x
> zu**SN<iMC51Q6Iza`SRnu5&nsNk|M!3m^T)PQSGWSb9rfX*>hVrMQFWP$B+vjPYX$
> z...@$5k3;D&_ZoYxt(Eu{9?UrQ3LFoEfa5?O;muzp8...@k{v-mo%_mp#fpnltlt{q
> zb8BWC<h>-3Gi*h42<07pIXyhNICX8D4utX%-|*VT1l~^^qN--0&W>-BI4ehZ0f&U@
> z;}BAPrNOqEg=U(F-r8WB%Qe&7dqy+Y2vODZuOA2&c...@qdt!^l@RTsGzuJTe9Uz$?
> zu%>}>ex7RsN6=;>=lLuV0$^KRei5ujxMZO5F2Ht+3m>Hn#F3|9t4NkZejGls#W2T>
> zJ9H+7eB%b&Vzbw8?Bd3|OK!aT%#BMM5~!8yXH^u?=$V{hEd84(8Pn!^yVL8ow_YLd
> zz{}t}E&4G}IJNFkv(s<)`YqPb&JuZdwva{TA1G;7iv2X2omZjBo=t7Kl(#J1E(g30
> zoV&9E&E7L;as&$I(OIOaOkMNlSldKU_In$&u...@z`cu59$8(vxkh!%2(e5)cKV{TZ
> z7<J3-xQ!yj4DO=L4-M&~u(EjZ>Z80=HBhX&=mCl&lo+FU=1yXJ+Y6aMLwnlZ&23x#
> zcDvu{uFz2DRk8QA68$0qZPC_dc37eg{<T*~gt)L&CbEGNa>3V1wvn2!;});aqoLm;
> zYxDEfLUr(%...@rsa+kur~tpt_6*$d0!28gstc_{g^tgsdf_rj=saao(e...@c%+9
> zm<xi04yWkM5OLwmEsj4Ir`L2cxaKY0<)yzb)y!4HvFrJ+m+A6<?J-=9t-f9_GS&A@
> zl5IRRSR(N_O_;h<s4hgxd7lv0tsqdm...@rr$0d0i`$cud`6osmt&C=dP|??JXPb?
> z$}XTA&Qckh=meWHVk3}ZH<k2bdWey(T!zU1?RJlI!rD_2V?&X^Q(TH1B|4LXb<K{_
> zPsG|yxnyeduz...@fdsih2gs5vx_y`{yj0f>HIa6iUHJPu-X<Zqe`>^...@*{r>*F
> z_MR5*T>9mSB^Yt{-naNVrON&^3Hs0X?OGixyLLlSY66;f...@ek4ez?uvpb<oqzkAM
> zq$y0}qt%-uh15x...@0?g@c4wPZ~k8Y}RK$Qm<0...@bydy}um+0e>l3zvwO)wa>;o
> z...@b{sekbpxdjupq;DK=92&LrwSpm>f...@w>Tt`gCH(DC)U6N{vdBiJ;izk&b#r
> z_ot$9;>C)muio4TXwR9#I(75e>gwN1XKibzijpAm;csse...@vlmmz%o-r_fefamb
> zWkoxgD>2IBq7=@@?1StI5DRTKGFn(z4qq...@tb2qhwuhn=<c97S+=uv)cMUbZa||
> zlKRBE5#)@pwirrux5q*+a%LT`)7OOiVKhf(x&}XoY7>U_mS;8SrTGfDz`D5C3Oo7u
> z?dQ`kM^}T>cbdh+^6}Nh=fmrF9bVO4LEkn9ICnC&YvzG{IS9W&3t159)Va3Ku1%px
> zMKMm-(%3wu)nnr|$EqJTFAje?)fmdgcuzyrjgq...@s+s#}p5#lo_7v3e+cxaj788
> zkh>;tl-co$-pK~w+X(7dvB-%Gy=I6*=MU?o-JQIWAEDJY?GtN3Cmww|7(%OFIy3Jr
> zn9SlAw$xiLs-ZKOAlFcmr)BoL3U}#AtIuK5hhqw`Y>T)W%HodX+?Xfs%)b...@tty+bu
> zcdu*kkr2s7wcmq...@onoc)o8yX7{+l9lFog)fw2)}lbtSGMC^SQqym&zR8jhcXf0
> zbT(k...@!kbiz`tqws56*tej9n;<@9(L|l{S38cu`=|r$VKhH$Urs8uJg9&)%H?=s!
> zBRFN7C#1V6Px&lsl}E_uW29P<F5X)VEhmoAumiiFE65_DX$~AkOdK}^lc1~eiG9$v
> zp;@6i)wv7-dNk_O-~I#s2f_~}z5iE>HY{&l1c5*C{80{-)I&gr12F~Do3Uf*oZ-ek
> zjgSuXkjhvy47ubzR5I~O&HeoG>hf1>>ejqdqqp02...@fj1_twk*++|Z|ZpdFxP)v
> zu5PGhQ+;-x(id3OqC9hHp@;y...@lw*dy<iCj1Z&CAQp_e_R<<+_TWkdd#R{6`mad=tF
> zRjKLT#+VF}^H3K`w&MX%=&NvTH&1+51bM$c&4&i~y6Fk(hpdzkg3...@gms8hdk9c
> z`jjBqPV}T)Hy{g>5e^AFXyj#{uj&%Q%;w+%I)3q26Z6q)={P3XoPVL=K8$lw=IC0+
> zID<ucOP^2#l=3$rmY{rGDS{T<i=AuS^Hpju)={(rprwld4jmraw=zjloo5...@p?y
> ZQU}woAN)!?mCXv3{{v;3Fh?07002HPG~56H
> 
> literal 0
> HcmV?d00001
> 
> diff --git a/rules/statserial.in b/rules/statserial.in
> new file mode 100644
> index 0000000..4d9a703
> --- /dev/null
> +++ b/rules/statserial.in
> @@ -0,0 +1,7 @@
> +## SECTION=shell_and_console
> +
> +config STATSERIAL
> +     tristate
> +     prompt "statserial"
> +     help
> +       Simple tool to monitor modem status lines on serial ports
> diff --git a/rules/statserial.make b/rules/statserial.make
> new file mode 100644
> index 0000000..51c493a
> --- /dev/null
> +++ b/rules/statserial.make
> @@ -0,0 +1,74 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2010 by konzeptpark GmbH
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_STATSERIAL) += statserial
> +
> +#
> +# Paths and names
> +#
> +STATSERIAL_VERSION   := 1.1.orig
> +STATSERIAL           := statserial-$(STATSERIAL_VERSION)
> +STATSERIAL_SUFFIX    := tar.gz
> +STATSERIAL_URL               := 
> http://archive.ubuntu.com/ubuntu/pool/universe/s/statserial/statserial_$(STATSERIAL_VERSION).$(STATSERIAL_SUFFIX)
> +STATSERIAL_SOURCE    := $(SRCDIR)/$(STATSERIAL).$(STATSERIAL_SUFFIX)
> +STATSERIAL_DIR               := $(BUILDDIR)/$(STATSERIAL)
> +STATSERIAL_LICENSE   := GPL
> +
> +# 
> ----------------------------------------------------------------------------
> +# Get
> +# 
> ----------------------------------------------------------------------------
> +
> +$(STATSERIAL_SOURCE):
> +     @$(call targetinfo)
> +     @$(call get, STATSERIAL)
> +     @ln -s statserial_$(STATSERIAL_VERSION).$(STATSERIAL_SUFFIX) 
> $(SRCDIR)/$(STATSERIAL).$(STATSERIAL_SUFFIX)

I don't like that. Have a look at mii-diag.make how to handle debian's
filenames.

> +
> +# 
> ----------------------------------------------------------------------------
> +# Prepare
> +# 
> ----------------------------------------------------------------------------
> +
> +#
> +# autoconf
> +#
> +STATSERIAL_CONF_TOOL := NO
> +
> +# 
> ----------------------------------------------------------------------------
> +# Compile
> +# 
> ----------------------------------------------------------------------------
> +
> +STATSERIAL_MAKE_ENV  := $(CROSS_ENV) \
> + LD=$(CROSS_CC) 
> +
> +# 
> ----------------------------------------------------------------------------
> +# Target-Install
> +# 
> ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/statserial.targetinstall:
> +     @$(call targetinfo)
> +
> +     @$(call install_init,  statserial)
> +     @$(call install_fixup, statserial,PACKAGE,statserial)
> +     @$(call install_fixup, statserial,PRIORITY,optional)
> +     @$(call install_fixup, statserial,VERSION,$(STATSERIAL_VERSION))
> +     @$(call install_fixup, statserial,SECTION,base)
> +     @$(call install_fixup, statserial,AUTHOR,"konzeptpark GmbH")
> +     @$(call install_fixup, statserial,DEPENDS,)
> +     @$(call install_fixup, statserial,DESCRIPTION,Simple tool to monitor 
> modem status lines on serial ports)
> +
> +     @$(call install_copy, statserial, 0, 0, 0755, 
> $(STATSERIAL_DIR)/statserial, /usr/bin/statserial)

please use "-" as source sheme here.
> +
> +     @$(call install_finish, statserial)
> +
> +     @$(call touch)
> +
> +# vim: syntax=make

cheers, Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachment: signature.asc
Description: OpenPGP digital signature

-- 
ptxdist mailing list
ptxdist@pengutronix.de

Reply via email to