[#9722] Kernel#system broken inside Dir.chdir(&block) if system command doesn't have shell characters — <noreply@...>

Bugs item #7278, was opened at 2006-12-14 13:59

8 messages 2006/12/14

[#9749] System V IPC in standard library? — Steven Jenkins <steven.jenkins@...>

Back in August, I needed a semaphore to serialize access to an external

14 messages 2006/12/19

[#9753] CVS freeze — SASADA Koichi <ko1@...>

Hi,

20 messages 2006/12/20
[#9755] Re: [ruby-dev:30039] CVS freeze — SASADA Koichi <ko1@...> 2006/12/20

Hi,

[#9757] Re: [ruby-dev:30040] Re: CVS freeze — SASADA Koichi <ko1@...> 2006/12/20

Hi,

Re: [ ruby-Bugs-7224 ] net/ftp login appears to be broken

From: Daniel Berger <djberg96@...>
Date: 2006-12-12 03:31:44 UTC
List: ruby-core #9707
noreply@rubyforge.org wrote:
> Bugs item #7224, was opened at 2006-12-12 13:34
> You can respond by visiting: 
> http://rubyforge.org/tracker/?func=detail&atid=1698&aid=7224&group_id=426
> 
> Category: Standard Library
> Group: 1.8.x
> Status: Open
> Resolution: None
> Priority: 3
> Submitted By: Robert Postill (robertpostill)
> Assigned to: Nobody (None)
> Summary: net/ftp login appears to be broken
> 
> Initial Comment:
> The impression I got from the documentation for net/ftp is that it should be possible to do the following:
> Net::FTP.new do |ftp|
>       ftp.login(hostname,username,password)
>       # Uncomment this line to see the FTP connection in action.
>       # ftp.debug_mode=true
>       for file in @files do #@files being an array of files
>         ftp.putbinaryfile(file)
>       end
> end

As I commented on RubyForge, FTP.open takes a block, FTP.new does not. 
I've made the same mistake myself.  But, it *is* correctly documented.

My confusion stems from the fact that most Ruby libraries tend to make 
'open' and 'new' synonymous (or nearly so), and they usually both take a 
block.

That being said, is there any reason we shouldn't have FTP.new yield 
self if a block is given? It's an easy, backwards compatible change, right?

Regards,

Dan

In This Thread

Prev Next