Hi,

On Wed, Sep 26, 2012 at 03:56:50PM +0200, Wolf Erik wrote:
> i try to send my first (small) patch, i hope it works. 
> 
> How can I create patches like the format from wolfram sangs last patch? I was
> only able to do it in the fomat below... :( 

This is done with "git send-email".

> >From 2bb361c3519c3c2c0f69848f4020f6c5f83791d8 Wed, 26 Sep 2012 15:48:46 +0200
> From: Erik Wolf <[email protected]>
> Date: Wed, 26 Sep 2012 15:48:05 +0200
> Subject: [PATCH] schedtool: add package

Don't forget your "Signed-off-by:"

> diff --git a/rules/schedtool.in b/rules/schedtool.in
> new file mode 100644
> index 0000000..0a814f5
> --- /dev/null
> +++ b/rules/schedtool.in
> @@ -0,0 +1,7 @@
> +## SECTION=shell_and_console
> +
> +config SCHEDTOOL
> +     tristate
> +     prompt "schedtool"
> +     help
> +       schedtool can be used to query or alter a process' scheduling policy 
> under linux
> diff --git a/rules/schedtool.make b/rules/schedtool.make
> new file mode 100644
> index 0000000..afc802f
> --- /dev/null
> +++ b/rules/schedtool.make
> @@ -0,0 +1,59 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2012 by Martin Wagner <[email protected]>
> +#
> +# 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_SCHEDTOOL) += schedtool
> +
> +#
> +# Paths and names
> +#
> +SCHEDTOOL_VERSION    := 1.3.0
> +SCHEDTOOL_MD5                := 0d968f05d3ad7675f1f33ef1f6d0a3fb
> +SCHEDTOOL            := schedtool-$(SCHEDTOOL_VERSION)
> +SCHEDTOOL_SUFFIX     := tar.bz2
> +SCHEDTOOL_URL                := 
> http://freequaos.host.sk/schedtool/$(SCHEDTOOL).$(SCHEDTOOL_SUFFIX)
> +SCHEDTOOL_SOURCE     := $(SRCDIR)/$(SCHEDTOOL).$(SCHEDTOOL_SUFFIX)
> +SCHEDTOOL_DIR                := $(BUILDDIR)/$(SCHEDTOOL)
> +SCHEDTOOL_LICENSE    := GPLv2
> +
> +# 
> ----------------------------------------------------------------------------
> +# Prepare
> +# 
> ----------------------------------------------------------------------------
> +
> +SCHEDTOOL_CONF_TOOL  := NO
> +SCHEDTOOL_MAKE_ENV   := $(CROSS_ENV)

add CPPFLAGS="$(CROSS_CFLAGS) $(CROSS_CPPFLAGS)" here. Otherwise the
CROSS_CFLAGS set by ptxdist are missing (e.g. PTXCONF_TARGET_EXTRA_CFLAGS).

I know this is not easy to spot. So for testing things like this I usually
have:
PTXCONF_TARGET_EXTRA_CPPFLAGS="-Iptx-cppflags"
PTXCONF_TARGET_EXTRA_CFLAGS="-Iptx-cflags"
PTXCONF_TARGET_EXTRA_CXXFLAGS="-Iptx-cxxflags"
PTXCONF_TARGET_EXTRA_LDFLAGS="-Lptx-ldflags"

to see which flags are really added to the command line.

> +SCHEDTOOL_MAKE_OPT   := CC=$(CROSS_CC) 

Check your white spaces.

SCHEDTOOL_INSTALL_OPT  := DESTPREFIX=/usr install

and then ...

> +
> +$(STATEDIR)/schedtool.prepare:
> +     @$(call targetinfo)
> +     @$(call touch)
> +
> +# 
> ----------------------------------------------------------------------------
> +# Target-Install
> +# 
> ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/schedtool.targetinstall:
> +     @$(call targetinfo)
> +
> +     @$(call install_init, schedtool)
> +     @$(call install_fixup, schedtool,PRIORITY,optional)
> +     @$(call install_fixup, schedtool,SECTION,base)
> +     @$(call install_fixup, schedtool,AUTHOR,"Martin Wagner 
> <[email protected]>")
> +     @$(call install_fixup, schedtool,DESCRIPTION,missing)
> +

... change this here:

        @$(call install_copy, schedtool, 0, 0, 0755, -, /usr/bin/schedtool)

Michael

> +
> +     @$(call install_finish, schedtool)
> +
> +     @$(call touch)
> +
> +# vim: syntax=make
> 
>  
>  
>  
> Mit freundlichen Grüßen
>  
> i.A. Erik Wolf
> Hardwareentwicklung 
>  
> Neuberger Gebäudeautomation GmbH
> Oberer Kaiserweg 6
> 91541 Rothenburg o.d.T.
>  
> Telefon:  +49 9861 402-542
> Telefax:  +49 9861 402-533
> E-Mail:   [email protected]
> Internet: www.neuberger.net
>  
> Handelsregister: AG Ansbach HRB 4933
> Sitz: Rothenburg ob der Tauber
> Geschäftsführer: Klaus Lenkner
> Ust.IdNr. DE 271 188 426
> 
> 
> -- 
> ptxdist mailing list
> [email protected]
> 

-- 
ptxdist mailing list
[email protected]

Reply via email to