Re: [osd-dev] [PATCH 10/15 ver2] tgt: os.h: getnameinfo() is different on none-Linux systems

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

 



On Thu, 19 Mar 2009 08:41:20 +0200
Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote:

> FUJITA Tomonori wrote:
> > On Thu, 12 Mar 2009 13:17:09 +0200
> > Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote:
> > 
> >> FUJITA Tomonori wrote:
> >>> Your logic is completely crazy to me. I will not merge your work if
> >>> you don't change your way.
> >>>
> >> This is your prerogative, in which case I will have to make my decisions.
> >>
> >>>> But there's never been a problem. This code works for years just fine.
> >>>> The problem at hand is BSD compatibility. As I said changes should be
> >>>> separate, cleanups is one thing, BSD code another. Most of the patches
> >>>> are BSD only stuff why can't they go in, they do nothing to the code.
> >>> The logic is broken. Working code doesn't mean that we don't need to
> >>> improve it.
> >>>
> >> I never said that, I was talking about how to improve it, you answer to
> >> things I never said, perhaps you misunderstood.
> > 
> > Huh?
> > 
> > What you said is: adding the compatibility code first and then working
> > on it (might remove some of them, etc).
> > 
> > I will not add any crazy compatibility code, which can be removed in
> > the future.
> 
> I hope there is none left. Do you see something else I should fix?

I'm not sure. That's why I needed to spend my time to fix two of your
patches! It's your job to convince me that your patch is definitely
necessary. Just resending your previous patchset based on your logic
that I can't accept is not useful.


--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux