Re: [PATCH] Re: linux-2.6.19-rc5-g088406bc build #120 failed

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

 



Krzysztof Halasa wrote:
We were in a perpetual state of:

1. supply patch
2. get criticism from new person just joining thread
3. change patch to address criticism
4. goto #1


Right, the description fits. OTOH I recall the criticism had a fair
amount of merit. That's how the things work here. Been there many
times BTW.

To be more precise, that is many distinct
criticisms from distinct people, some of which
contradict each other.

I know code is open to criticism,
but after several weeks of submitting patches
and getting no closer to acceptance I gave up.
We were going around in circles where one person
wanted some thing that conflicted with what
another person wanted.

But since we seem stuck in a state where real fixes
are not allowed, and this breakage is constantly reintroduced,

It may look like that sometimes but it's not real. Anyway I think
everyone would benefit from the correct fix and the issue wouldn't
come again. Having looked at it I'd fix it myself but I'm pretty
sure you still have the old patch (which changes CONFIG_ macros
outside Kconfig, I mean in .c files) and it could be trivially
modified then applied (and perhaps tested with real hardware if
needed).

There isn't much point in resubmitting the previously
rejected patches only to be rejected again.

I was planning on trying again in a month or two
to see if all the people who rejected all of the
previous patches have come to some sort of agreement.

Without such agreement we are left in the state
of never ending patches.

--
Paul Fulghum
Microgate Systems, Ltd.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux