Geert Stappers

Results 419 comments of Geert Stappers

Commit 54c4c1d is an improvement. The build now fails with: ```text crypto/deflate.c: In function ‘deflate_inflate’: crypto/deflate.c:514:41: error: storing the address of local variable ‘zlib_header’ in ‘*deflate.resume’ [-Werror=dangling-pointer=] 514 | deflate->resume...

While building with #820 I learnt there is commit [b0ded89e917](https://github.com/ipxe/ipxe/commit/b0ded89e917b48b73097d3b8b88dfa3afb264ed0) that _disables dangling pointer checking for GCC_. So I get clean compiles with the main branch. When I test the...

> > attempt the bisect. Cloning V1.0.0 failed to compile, as did most of the bisect points. I can compile the current version. > Newer compilers tend to be more...

> There are instructions at https://ipxe.org/howto/bisect that walk you through the process. Replace the _good_ from that webpage with the _good_ from the top of this page, like: ```text git...

> if you are able to compile I suggest to do `make clean` and another `make NO_WERROR=1`.

On Mon, Jan 23, 2023 at 12:20:42PM -0800, Rob Mahn wrote: > I do wonder why you like the menu positioned at the bottom of the screen. Bottom of screen...

Cross reference to [another dangling patch](https://github.com/ipxe/ipxe/discussions/721#discussioncomment-4690014).

> This is related to #883 And #883 is closed by #968 about a month ago. Highlighting @rumline for seeing this feedback request.

FYI: mcb30 linked a pull request 8 minutes ago that will close this issue (upon merge).

Knowing that a there are several "`intel*.c`" files. ```text stappers@paddy:~/src/githubforks/ipxe $ ls src/drivers/net/intel*c src/drivers/net/intel.c src/drivers/net/intelx.c src/drivers/net/intelxlvf.c src/drivers/net/intelvf.c src/drivers/net/intelxl.c src/drivers/net/intelxvf.c stappers@paddy:~/src/githubforks/ipxe $ ``` I started with `git grep -l 8086` to...