<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 8, 2013 at 2:39 PM, Greg Hellings <span dir="ltr"><<a href="mailto:greg.hellings@gmail.com" target="_blank">greg.hellings@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks! I updated CMake as we talked about.<br>
<br>
The current SVN HEAD of Xiphos now has support for adding SFTP sources<br>
through its module manager. Shout-out to Karl (or whoever wrote that<br>
dialog) for making the code there very straightforward. Just adding a<br>
source with type "SFTP" in Xiphos' module manager will create the<br>
necessary entry in InstallMgr.conf and any SFTP sources that SWORD<br>
reports will act normally.<br></blockquote><div><br></div><div style>I've also added the ability to create/modify HTTP, HTTPS and SFTP transports to BibleTime, although it still lacks the ability to specify a username or password. Support can be found in the branch sword-svn-compat from the BibleTime repositories. However, it probably will not compile if your build of SWORD has Troy's check in from earlier this morning.</div>
<div style><br></div><div style>--Greg</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Note that SWORD will silently and harmlessly ignore any<br>
InstallMgr.conf entries that are SFTPSource entries if it was compiled<br>
without SFTP support.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Greg<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Sun, Jan 6, 2013 at 1:29 PM, Troy A. Griffitts <<a href="mailto:scribe@crosswire.org">scribe@crosswire.org</a>> wrote:<br>
> Applied. Thanks Greg, for the original contribution and for taking the time<br>
> to work through the details.<br>
><br>
><br>
> On 12/31/2012 03:53 PM, Greg Hellings wrote:<br>
><br>
> Here is the updated patch adding CURLSFTPAVAILABLE support to CMake as<br>
> well as to the library code. It defaults to assuming no SFTP support<br>
> if either the curl-config executable cannot be found or if it returns<br>
> a value other than "1" from the quick detection process we have<br>
> settled on.<br>
><br>
> Users of Windows builds through either VisualStudio or Borland will<br>
> need to figure out if curl-config is available on their systems and,<br>
> if not, come up with another way to detect and support SFTP for<br>
> clients using those builds. On Linux it is a Bash script, so it might<br>
> be adaptable to Manfred's XCode system in some way also.<br>
><br>
> For those in the JSword world, I am sure there are SFTP Java clients<br>
> available which could be leveraged if they wanted to add support for<br>
> the same functionality to JSword applications.<br>
><br>
> --Greg<br>
><br>
> On Sun, Dec 30, 2012 at 9:11 PM, Troy A. Griffitts <<a href="mailto:scribe@crosswire.org">scribe@crosswire.org</a>><br>
> wrote:<br>
><br>
> OK Greg,<br>
><br>
> I've hacked detection of CURL SFTP into the autotools build (hopefully-- it<br>
> works for me).<br>
><br>
> I've added a new compile time define with -DCURLSFTPAVAILABLE to go along<br>
> with the existing -DCURLAVAILABLE<br>
><br>
> If you'd like to update the sftp patch to conditionally compile support in<br>
> based on this define, that would be cool. I'll do it myself soon if you<br>
> don't have time.<br>
><br>
> Troy<br>
><br>
><br>
><br>
> On 12/28/2012 11:42 AM, Greg Hellings wrote:<br>
><br>
> Further digging with help from our friends has revealed this nugget:<br>
><br>
> $ curl-config --protocols<br>
><br>
> produces a newline-delimited list of protocols that the particular<br>
> build of libcurl supports. curl-config is a shell script which can be<br>
> run on the build system and should satisfy both the requirements of<br>
> native builds and cross-compiling support. We could use this to set a<br>
> compiler macro indicating support (or not) for SFTP in the target<br>
> libcurl library.<br>
><br>
> If someone wants to tackle that in the autotools world, I can add<br>
> detection to CMake as well. A simple command such as<br>
> $ curl-config --protocols | grep SFTP | wc -l<br>
> 1<br>
><br>
> in Fedora will result in a value of 1 or greater if SFTP is supported<br>
> while it should produce 0 if SFTP support is left out. An Ubuntu<br>
> system produce this output:<br>
> $ curl-config --protocols | grep SFTP | wc -l<br>
> 0<br>
><br>
> And it even works for cross-compiling:<br>
> $ /usr/i686-w64-mingw32/sys-root/mingw/bin/curl-config --protocols |<br>
> grep SFTP | wc -l<br>
> 1<br>
><br>
><br>
> This appears to be our best way forward if we want to enable<br>
> compile-time enabling or disabling of this option.<br>
><br>
> --Greg<br>
><br>
> On Mon, Dec 24, 2012 at 8:43 AM, Greg Hellings <<a href="mailto:greg.hellings@gmail.com">greg.hellings@gmail.com</a>><br>
> wrote:<br>
><br>
> Troy,<br>
><br>
> On Sun, Dec 23, 2012 at 10:39 PM, Troy A. Griffitts<br>
> <<a href="mailto:scribe@crosswire.org">scribe@crosswire.org</a>> wrote:<br>
><br>
> Dear Greg,<br>
><br>
> Looking to apply this SFTP patch, could you give me some background as<br>
> to<br>
> why the check to ignore across all transports for '.' and '..'?<br>
><br>
> Our downloading method runs recursively from the given directory until<br>
> it runs out of directory depth. FTP servers don't usually seem to<br>
> return . and .. as valid paths, and the HTTP(S) transport attempts to<br>
> parse the returned HTML page to avoid the link to the parent<br>
> directory. But whatever options are passed by cURL to the SFTP<br>
> transport resulted in it returning . and .. as paths within the<br>
> current directory. Because '.' came first in the list, the installmgr<br>
> was running through an infinite loop whenever it tried to pull data<br>
> from the server.<br>
><br>
> I added it at the level of all transports because we don't want to<br>
> either loop infinitely on '.' or accidentally pull a whole server<br>
> recursively by following '..' to the root of the server. It might be a<br>
> server config option that permits it, but I wanted to avoid the<br>
> possibility of the InlstallMgr class getting choked up on it.<br>
><br>
> --Greg<br>
><br>
> Thanks,<br>
><br>
> Troy<br>
><br>
><br>
><br>
><br>
> On 12/03/2012 04:06 PM, Greg Hellings wrote:<br>
><br>
> The attached patch will introduce support for SFTPSource transports in<br>
> the SWORD engine, allowing a user to access remote repositories over<br>
> SFTP (which is enabled by default when a user enables SSH).<br>
><br>
> --Greg<br>
><br>
><br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
><br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
><br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
><br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
</div></div></blockquote></div><br></div></div>