xfs
[Top] [All Lists]

Re: [PATCH v3 1/2] xfstests: add fssum tool

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH v3 1/2] xfstests: add fssum tool
From: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date: Sun, 11 Aug 2013 21:54:41 -0500
Cc: Jan Schmidt <list.xfs@xxxxxxxxxxxxx>, "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx>, "sbehrens@xxxxxxxxxxxxxxxx" <sbehrens@xxxxxxxxxxxxxxxx>, "linux-btrfs@xxxxxxxxxxxxxxx" <linux-btrfs@xxxxxxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130812011539.GM12779@dastard>
References: <1375949833-1104-1-git-send-email-list.xfs@xxxxxxxxxxxxx> <1375949833-1104-2-git-send-email-list.xfs@xxxxxxxxxxxxx> <5203D827.3050909@xxxxxxxxxxx> <20130812011539.GM12779@dastard>
On Aug 11, 2013, at 8:15 PM, Dave Chinner <david@xxxxxxxxxxxxx> wrote:

> On Thu, Aug 08, 2013 at 12:40:55PM -0500, Eric Sandeen wrote:
>> On 8/8/13 3:17 AM, Jan Schmidt wrote:
>>> SUBDIRS =
>>> 
>>> LLDLIBS = $(LIBATTR) $(LIBHANDLE) $(LIBACL)
>>> 
>>> +OPT_LDLIBS = -lssl -lcrypto
>> 
>> Hm, new deps.  I guess it's not a huge problem, these should always
>> be available, right?
> 
> Please, let's keep crypto out of xfstests if we can. That's just
> going to add a nightmare of US export compliance garbage to any
> distro that wants to package and ship this....
> 
It's ssl that's (optionally) required for check summing I believe.  But it's 
optional if done properly...

-Eric

> Cheers,
> 
> Dave.
> -- 
> Dave Chinner
> david@xxxxxxxxxxxxx
> 

<Prev in Thread] Current Thread [Next in Thread>