[PATCH] Log feature: Correct bad function implementation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>-----Original Message-----
>From: Colin Guthrie [mailto:gmane at colin.guthr.ie]
>Sent: Thursday, March 24, 2011 12:02 PM
>To: General PulseAudio Discussion
>Cc: Becker, VincentX
>Subject: Re: [PATCH] Log feature: Correct bad function implementation
>
>'Twas brillig, and Vincent Becker at 24/03/11 10:35 did gyre and gimble:
>> Replace wrong implementation of log to file in
>> pa_daemon_conf_set_log_level to pa_daemon_conf_set_log_target
>
>Sorry but this is not based on git master.... (as Maarten asked for
>before).
>
>[colin at jimmy pulseaudio (master|AM)]$ cat ~/Download/pa.patch | patch
>-p1 --dry-run
>patching file src/daemon/daemon-conf.c
>Hunk #1 FAILED at 142.
>Hunk #2 FAILED at 170.
>Hunk #3 succeeded at 187 (offset -5 lines).
>Hunk #4 FAILED at 235.
>3 out of 4 hunks FAILED -- saving rejects to file src/daemon/daemon-
>conf.c.rej
>
>I think it's just a matter of ignoring hunks 1 & 2 anyway (as I already
>made that change when I committed the original version) and the move
>from 4 to 3 should just be updated as the code in hunk 4 was updated
>(tho' the newer code is the same as you put in in your hunk 3).
>

Sorry I am still learning with git. I thought that taking the code as-is was enough. I have just sent a patch back to you from master branch, and it should be OK, I hope.

Vin

>IOW, feel free to update your patch, or alternatively, I can just commit
>what I have..... (straight move of the code) which I've attached. Let me
>know if it's OK.
>
>
>Cheers
>
>Col
>
>PS, kinda embarrassed I missed this glaring error first time round :D
>
>
>
>--
>
>Colin Guthrie
>gmane(at)colin.guthr.ie
>http://colin.guthr.ie/
>
>Day Job:
>  Tribalogic Limited [http://www.tribalogic.net/] Open Source:
>  Mageia Contributor [http://www.mageia.org/]
>  PulseAudio Hacker [http://www.pulseaudio.org/]
>  Trac Hacker [http://trac.edgewall.org/]
---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux