search

kenahoo / Path-Class

Cross-platform path specification manipulation
http://search.cpan.org/dist/Path-Class/
15 stars 28 forks source link

spew_lines with test #30

Closed skybluecircles closed 10 years ago

skybluecircles commented 10 years ago

This PR creates a new method called spew_lines - it's just like spew, but if a plain scalar is passed, it appends $/ to that scalar. Otherwise, if an array ref is passed, appends $/ to each element of the array.

If spew is the opposite of slurp, spew_lines would be the opposite of slurp( chomp => 1 )

Documentation and a test are included.

skybluecircles commented 10 years ago

The test suite passed when I ran it locally, but it looks like the Travis build failed - is there any thing I need to do to get it pass?

kenahoo commented 10 years ago

Hi William,

I finally got Travis working, I think. Could you merge or rebase your pull request on top of master? I think it would pass now.

skybluecircles commented 10 years ago

Hi Ken,

I rebased off of the upstream master - it looks like Travis passed. I think we're good to go?

kenahoo commented 10 years ago

Great, thanks for sticking with it! I merged the PR.

skybluecircles commented 10 years ago

Sure, np - thanks! (And thanks for creating Path::Class ;) )

mrdvt92 commented 10 years ago

Path::Class Folks, I really like Path::Class and have integrated a Secure FTP client with Path::Class::File as the return object. If anyone is interested you should check it out. Net-SFTP-Foreign-Tempdir-Extract

I sub classed Path::Class in Net::SFTP::Foreign::Tempdir::Extract::File to add an "extract" method (currently only supports ZIP files). But, Let me know if this is something that might be more universally integrated into Path::Class itself or a package in the Path::Class::Extract name space. Thanks, Mike