Bug 42756 - Read support patch for Poi-Ruby
Summary: Read support patch for Poi-Ruby
Status: RESOLVED LATER
Alias: None
Product: POI
Classification: Unclassified
Component: HSSF (show other bugs)
Version: unspecified
Hardware: Other other
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-27 12:31 UTC by Adam Lebsack
Modified: 2013-08-04 19:59 UTC (History)
0 users



Attachments
The patch (4.38 KB, patch)
2007-06-27 12:32 UTC, Adam Lebsack
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Lebsack 2007-06-27 12:31:22 UTC
I needed read support for Poi-Ruby, so I created it....  however this form does not accept file uploads for 
my patch.  You may contact me at my email address for the patch, unless I find a way to attach it to this 
issue.
Comment 1 Adam Lebsack 2007-06-27 12:32:12 UTC
Created attachment 20404 [details]
The patch
Comment 2 Nick Burch 2007-12-04 04:44:35 UTC
Your patch appears to be missing the new file
"org/apache/poi/RubyInputStream.h". Any chance you could upload it so we can
apply the patch?
Comment 3 David Fisher 2009-02-18 11:49:16 UTC
Thanks again for submitting this patch. But as it is it useless to apply it.

More than the header is missing. Here is the current poi-ruby directory structure from POI.

.
./java/org/apache/poi/RubyOutputStream.java
./cpp
./cpp/RubyIO.cpp
./tests
./tests/ts_all.rb
./tests/tc_gc.rb
./tests/tc_base_tests.rb
./poi-2.0-final-20040126.jar
./Poi4R.i
./Makefile

I think what we'll need

(1) ./java/org/apache/poi/RubyInputStream.java
(2) ./cpp/RubyInputStream.h
(3) ./cpp/RubyOutputStream.h
(4) A more modern version of POI!
(5) A test of the input stream.

HTH, being "Java" developers, I don't how many of us are ruby people. Make it easy for us, and we'll do what we can to add the support.
Comment 4 Dominik Stadler 2013-08-04 19:59:54 UTC
Setting to RESOLVED after a long time without update.