public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
From: Phil Muldoon <pmuldoon@redhat.com>
To: frysk@sourceware.org
Subject: Re: [SCM]  master: Fix changelog to ElfPrstatus.cxx, and really commit  ElfPrstatus.cxx import change from asm/user.h to sys/user.h.
Date: Mon, 07 Apr 2008 14:11:00 -0000	[thread overview]
Message-ID: <47FA22EC.6030603@redhat.com> (raw)
In-Reply-To: <20080407133107.2891.qmail@sourceware.org>

pmuldoon@sourceware.org wrote:

This commit tweaks the imports of ElfPrStatus.cxx to import 
/usr/include/sys/user.h instead of /use/include/asm/user.h. As Petr 
pointed out to me, the kernel-headers on rawhide do not carry the user.h 
file anymore (not sure of the reason). Potentially the file should never 
have included a file from asm/ as it is. So tweaked commit to include 
the user.h provided by: glibc-headers

Regards

Phil
> The branch, master has been updated
>        via  fd38b34dcc23843a0dd48d914878a6c9b9418bae (commit)
>       from  4d2cf9c645d3507028490b6eec92a381c228ffc0 (commit)
>
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email.
>
> - Log -----------------------------------------------------------------
> commit fd38b34dcc23843a0dd48d914878a6c9b9418bae
> Author: Phil Muldoon <pmuldoon@redhat.com>
> Date:   Mon Apr 7 14:30:19 2008 +0100
>
>     Fix changelog to ElfPrstatus.cxx, and really commit ElfPrstatus.cxx import change from asm/user.h to sys/user.h.
>     
>     2008-04-07  Phil Muldoon  <pmuldoon@redhat.com>
>     
>             * cni/ElfPrstatus.cxx: Delete not needed
>             asm/elf.h import. Change asm/user.h to
>             sys/user.h.
>
> -----------------------------------------------------------------------
>
> Summary of changes:
>  frysk-sys/lib/dwfl/ChangeLog           |    2 +-
>  frysk-sys/lib/dwfl/cni/ElfPrstatus.cxx |    4 +---
>  2 files changed, 2 insertions(+), 4 deletions(-)
>
> First 500 lines of diff:
> diff --git a/frysk-sys/lib/dwfl/ChangeLog b/frysk-sys/lib/dwfl/ChangeLog
> index 1f7d226..9ba1f70 100644
> --- a/frysk-sys/lib/dwfl/ChangeLog
> +++ b/frysk-sys/lib/dwfl/ChangeLog
> @@ -1,6 +1,6 @@
>  2008-04-07  Phil Muldoon  <pmuldoon@redhat.com>
>  
> -	* cni/ElfPrPrinfo.cxx: Delete not needed
> +	* cni/ElfPrstatus.cxx: Delete not needed
>  	asm/elf.h import. Change asm/user.h to
>  	sys/user.h.
>  	
> diff --git a/frysk-sys/lib/dwfl/cni/ElfPrstatus.cxx b/frysk-sys/lib/dwfl/cni/ElfPrstatus.cxx
> index 3afe03f..36f60e4 100644
> --- a/frysk-sys/lib/dwfl/cni/ElfPrstatus.cxx
> +++ b/frysk-sys/lib/dwfl/cni/ElfPrstatus.cxx
> @@ -52,9 +52,7 @@ typedef __u32 u32;
>  #include "elf.h"
>  #include "libelf.h"
>  #include "gelf.h"
> -
> -#include "asm/elf.h"
> -#include "asm/user.h"
> +#include "sys/user.h"
>  #include "lib/dwfl/ElfPrstatus.h"
>  #include "lib/dwfl/ElfData.h"
>  #include "lib/dwfl/ElfException.h"
>
>
> hooks/post-receive
> --
> frysk system monitor/debugger
>   

           reply	other threads:[~2008-04-07 13:34 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20080407133107.2891.qmail@sourceware.org>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47FA22EC.6030603@redhat.com \
    --to=pmuldoon@redhat.com \
    --cc=frysk@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).