Are there any issues with this patch? Or can we push this?
It worked just fine for me. I guess, we should update the others too.
On Thu, Jun 12, 2014 at 8:53 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Donnerstag, 12. Juni 2014, 13:24:02 schrieb Giuseppe Scrivano:
Darshit Shah dar...@gmail.com
It is ok for me.
Here is slightly different approach to set ENABLE_PSL.
Maybe you find it more readable / understandable.
ENABLE_PSL=no
AC_ARG_WITH(libpsl,
AS_HELP_STRING([--without-libpsl], [disable support for libpsl cookie
checking.]),
[],
[AC_SEARCH_LIBS(psl_builtin, psl,
Am Mittwoch, 11. Juni 2014, 18:57:13 schrieb Darshit Shah:
On Wed, Jun 11, 2014 at 5:20 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
Darshit Shah dar...@gmail.com writes:
On Wed, Jun 11, 2014 at 5:20 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file
Yes, the configure statements given by Tim work. I found out that the issue on
machine was caching of configure values. Deleting the configure cache fixed the
issue.
I also agree with Giuseppe's point about not using the autoconf variables.
Let's fix the rest of them too.
Tim's patch however
Am Donnerstag, 12. Juni 2014, 07:16:13 schrieb Darshit Shah:
Yes, the configure statements given by Tim work. I found out that the issue
on machine was caching of configure values. Deleting the configure cache
fixed the issue.
I also agree with Giuseppe's point about not using the autoconf
Am Donnerstag, 12. Juni 2014, 13:24:02 schrieb Giuseppe Scrivano:
Darshit Shah dar...@gmail.com writes:
On Wed, Jun 11, 2014 at 5:20 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl
That patch works perfectly for me.
Maybe we should update the rest of the file along the same lines? It
would help to reduce some fragmentation.
On Thu, Jun 12, 2014 at 8:53 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Donnerstag, 12. Juni 2014, 13:24:02 schrieb Giuseppe Scrivano:
Darshit Shah
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file does not work as expected. When libpsl is not
installed on a system, the LDFLAGS does not contain -lpsl flag,
Am Mittwoch, 11. Juni 2014, 13:50:46 schrieb Tim Rühsen:
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file does not work as expected. When libpsl is not
On Wed, Jun 11, 2014 at 5:20 PM, Tim Rühsen tim.rueh...@gmx.de wrote:
Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file does not work as expected. When libpsl is
Hi Giuseppe,
Thanks for the patch. But no, it doesn't solve the problem yes.
Here is the configure summary I get:
configure: Summary of build options:
Version: 1.15.28-854eb-dirty
Host OS: linux-gnu
Install prefix:/usr/local
Compiler: clang
CFlags:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file does not work as expected. When libpsl is not
installed on a system, the LDFLAGS does not contain -lpsl flag, but
the configure summary shows LIBPSL: Yes.
There is some
Darshit Shah dar...@gmail.com writes:
Not sure about using pkg-config in Wget's configure.ac. That would be an
option for detecting libpsl (and other libs as well, I guess). We can work
also on that later if there are no complaints against that.
I'm not sure about using pkg-config. I don't
Darshit Shah dar...@gmail.com writes:
I'm facing an issue with the patch I submitted for libpsl and would be
glad if someone could help me.
The configure.ac file does not work as expected. When libpsl is not
installed on a system, the LDFLAGS does not contain -lpsl flag, but
the configure
Tim,
As the author of libpsl, I'm waiting on you to ACK this, so we can merge.
On Wed, Jun 4, 2014 at 4:30 PM, Giuseppe Scrivano gscri...@redhat.com wrote:
Darshit Shah dar...@gmail.com writes:
From 5b25217ecf6eb1897d769f2ee0aa5e922e6cbff4 Mon Sep 17 00:00:00 2001
From: Darshit Shah
On Thursday 05 June 2014 15:27:21 Darshit Shah wrote:
Tim,
As the author of libpsl, I'm waiting on you to ACK this, so we can merge.
Sorry for letting you wait, Darshit.
The patch looks good to me though i am not able to test it right now. But i am
sure, you did it already ;-)
Not correctly
On Thu, Jun 5, 2014 at 4:16 PM, Tim Ruehsen tim.rueh...@gmx.de wrote:
On Thursday 05 June 2014 15:27:21 Darshit Shah wrote:
Tim,
As the author of libpsl, I'm waiting on you to ACK this, so we can merge.
Sorry for letting you wait, Darshit.
Sure, no issues.
The patch looks good to me though
Hi,
I've updated the patch with a small change to add libpsl to the list
of not absolutely required dependencies.
On Sun, Jun 1, 2014 at 2:35 AM, Ángel González keis...@gmail.com wrote:
On 31/05/14 21:30, Darshit Shah wrote:
+ * [35]libpsl is (optionally) required for checking cookie
Am Freitag, 30. Mai 2014, 22:24:19 schrieb Darshit Shah:
I've attached a patch that adds support for using libpsl for cookie
domain checking in Wget.
The old heuristic checks still remain as a fallback. When the libpsl
library on the system is built without the builtin list, Wget simply
Hi Tim,
I've already submitted an update version of the patch to match the new library
name. Do let me know if you haven't received it. I'll resend the email in that
case.
—
Thanking you,
Darshit Shah
Sent from mobile. Please excuse any errors.
On Sun, Jun 1, 2014 at 3:46 PM, Tim Rühsen
Hi,
This is an updated version of the patch to build against the latest
libpsl release which changes the so-numbers.
P.S.: In case anyone is trying to test this on a Arch Linux system,
I've uploaded libpsl to the AUR.
For users of debian and derivative systems, I think there's a package
coming
On Sun, Jun 1, 2014 at 2:35 AM, Ángel González keis...@gmail.com wrote:
On 31/05/14 21:30, Darshit Shah wrote:
+ * [35]libpsl is (optionally) required for checking cookie domains.
+
It's weird to have something optionally required. I would remove
(optionally)
and add it at the list on
I've attached a patch that adds support for using libpsl for cookie
domain checking in Wget.
The old heuristic checks still remain as a fallback. When the libpsl
library on the system is built without the builtin list, Wget simply
fallsback to the old heuristic checks. Similarly, if wget is built
24 matches
Mail list logo