James Stronz

Results 18 comments of James Stronz

Can someone please merge this? The fix does work and saved me from having to use archlinux-lts.

@quanium-89 Afaik, #64 changes are consistent with changes to `net/core/dev.c` From 5.11 ``` case GRO_DROP: kfree_skb(skb); break; case GRO_MERGED_FREE: if (NAPI_GRO_CB(skb)->free == NAPI_GRO_FREE_STOLEN_HEAD) napi_skb_free_stolen_head(skb); else __kfree_skb(skb); break; ``` to master...

> It definitely broke all my `.python-version` files. But if I understand correctly this was a CVE fix? I can work around it but would love for the old behavior...

@native-api that is one way. In terms of compatibility, would using `realpath` or `readlink -f` be a valid choice? I'm debating if your normalization proposal is reinventing realpath/readlink again.

@native-api I opened a PR for this issue. I'll work on fixing the failing tests soon.

That fix turned out to be a bit trickier than I initially thought. Namely, the unit testing had to be updated to satisfy the preconditions of the `is_version_safe` function (it...

Sorry for all of these commit references due to me amending issues—I am 💯 done and ready for any review comments. "All checks have passed" for PR #2442.

@basepi awesome! @native-api, my changes after @basepi's review were to remove accidental complexity, rewrite comments, and reduce how much you have to review! _"The height of sophistication is simplicity."_

@basepi @native-api I added some output for when we ignore version names. Let me know if there is anything else.

I think an existing issue or maybe implementation mistake in my spec results in: ``` [492060] PyInstaller Bootloader 3.x [492060] LOADER: executable is /path/to/daemons/webserver/webserver [492060] LOADER: homepath is /path/to/daemons/webserver [492060]...