I am extremely sorry.
I raised the PR in ls-qpack's repo at 
https://github.com/litespeedtech/ls-qpack/pull/76

Thanks
Pranav

________________________________
From: Colin Watson <cjwat...@debian.org>
Sent: Thursday, March 20, 2025 6:19 PM
To: Pranav P <pranav...@ibm.com>
Cc: 1099935 <1099...@bugs.debian.org>; elbrus <elb...@debian.org>; debian-s390 
<debian-s...@lists.debian.org>
Subject: [EXTERNAL] Re: Bug#1099935: dnspython: autopkgtest regression on 
s390x: bad request

On Thu, Mar 20, 2025 at 07:17:10AM +0000, Pranav P wrote:
>Thanks a lot. I raised an issue with a suggested fix on pylsqpack's github repo

https://github.com/aiortc/pylsqpack/issues/38  , for reference.

>and has
>asked them how they want to integrate the changes. I will also raise a PR to 
>ls-qpack's
>repo. Meanwhile, I am attaching the patch with a commit message generated via 
>quilt.
>I am new to the community so please feel free to share if something is to be 
>modified.
>
>Thanks
>Pranav
>
>[https://res.public.onecdn.static.microsoft/assets/mail/file-icon/png/generic_16x16.png%5D0001-Fix-enddianness-bug-in-pylsqpack-Encoder.encode.patch
>  
><https://ibm-my.sharepoint.com/:u:/p/pranav_p7/EcY4vFeM7QpCnoEzfnuMXr0BYOU-u3vndDV3ESeVodixIA
>  >

This attachment wasn't an email attachment - it was some kind of
Sharepoint link that I can't access.  Could you please resend your
attachment as an actual email attachment, and then I can get it
into the Debian pylsqpack package?

Alternatively, pointing me at your ls-qpack PR once it exists would be
fine too - I don't see anything there yet.

Thanks,

--
Colin Watson (he/him)                              [cjwat...@debian.org]
From: Pranav P <pranav...@ibm.com>
Description: [PATCH] Fix endianness bug in pylsqpack Encoder.encode
 Fix endianness bug in big endian systems that arise because of
 comparison of macros __BYTE_ORDER and __LITTLE_ENDIAN which are
 defined in endian.h and is not added by default by c99 standard.
Bug: https://bugs.debian.org/1099935

--- a/vendor/ls-qpack/lsqpack.c
+++ b/vendor/ls-qpack/lsqpack.c
@@ -37,6 +37,7 @@
 #include <string.h>
 #include <sys/queue.h>
 #include <sys/types.h>
+#include <endian.h>
 #include <inttypes.h>
 
 #include "lsqpack.h"

Reply via email to