Messages in this thread | | | Date | Mon, 09 Jan 2006 08:42:06 -0500 | From | Ben Collins <> | Subject | Re: [PATCH 15/15] kconf: Check for eof from input stream. |
| |
On Mon, 2006-01-09 at 12:32 +0100, Roman Zippel wrote: > Hi, > > On Monday 09 January 2006 04:59, Ben Collins wrote: > > > > Then something is wrong with your automatic build. If the config needs to > > > be updated and stdin is redirected during a kbuild, it will already > > > abort. > > > > And what should be directed into stdin? Nothing. There should be no > > input going into an automated build, exactly because it could produce > > incorrect results. > > > > BTW, this is the automatic build that Debian and Ubuntu both use (in > > Debian's case, used for quite a number of years). So this isn't > > something I whipped up. > > That just means Debian's automatic build for the kernel has been broken for > years. All normal config targets require user input and no input equals > default input. Only silentoldconfig will abort if input is not available.
I think that's broken (because I don't see where that behavior is described). IMO, based on the code, it should only go with defaults when -n -y or -m is passed. Anything else should detect when stdin is not valid and abort. If stdin is valid, and empty string is read, then that's a valid "give me the default" response.
Why is it so hard to error when stdin is closed? It's not like that will break anything.
Since silentoldconfig does this, then oldconfig should aswell. The only naming difference is that silentoldconfig is quiet, and oldconfig is not. Why should the two act any differently with a closed stdin?
-- Ben Collins <ben.collins@ubuntu.com> Developer Ubuntu Linux
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |