search

hoijui / Jawk

Java AWK interpreter/compiler
GNU General Public License v3.0
39 stars 23 forks source link

Enhancement: Allow Jawk to print to an arbitrary OutputStream #11

Open bertysentry opened 6 years ago

bertysentry commented 6 years ago

When not printing to a file, AVM.interpret() will print to System.out. One may need to print to something else (Socket, ByteArrayOutputStream, etc.).

Add an option in the AwkSettings class to set the output stream.

The same probably needs to be done for compilation.

cdrasmussen commented 6 years ago

Are you running it standalone or inside another program like, say Java? I've redirected stdout from scripts in Windows cmd and Unix shell.

Maybe you should tell us more about the problem

+CDR

On Jan 27, 2018 7:13 AM, "Bertrand Martin" notifications@github.com wrote:

When not printing to a file, AVM.interpret() will print to System.out. One may need to print to something else (Socket, ByteArrayOutputStream, etc.).

Add an option in the AwkSettings class to set the output stream.

The same probably needs to be done for compilation.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hoijui/Jawk/issues/11, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAU9Apsm89ajKEIutdwCZQXK5HSfeK6ks5tOxL0gaJpZM4RvRIu .

eyalroth commented 6 years ago

I think this repository is pretty much dead now. I tried to revive it with PR #10 but I didn't get any response from the owner @hoijui.

I actually have plenty more fixes and features working for me locally -- including the one you just asked for -- but seeing how the repository is dead I gave up on contributing them.

cdrasmussen commented 6 years ago

Ok it's open source and it's a good project. So for it and become the new maintainer. I'd like to see the functions added that Gawk

+CDR

On Jan 27, 2018 3:55 PM, "Eyal Roth" notifications@github.com wrote:

I think this repository is pretty much dead now. I tried to revive it with PR #10 https://github.com/hoijui/Jawk/pull/10 but I didn't get any response from the owner @hoijui https://github.com/hoijui.

I actually have plenty more fixes and features working for me locally -- including the one you just asked for -- but seeing how the repository is dead I gave up on contributing them.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/hoijui/Jawk/issues/11#issuecomment-361014584, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAU9P6kTCwzY1WvVy0eq3tmk-v00nrRks5tO41IgaJpZM4RvRIu .

bertysentry commented 6 years ago

@eyalroth I see you forked the Jawk repo, but it looks like you only pushed your changes to fix the bug "Compiled Awk can't read input". Would you mind sharing your bug fixes and enhancements? (just pushing to your repo). If needed, I'd be happy to contribute if I see fit. This way, you'd become the new official maintainer of Jawk ;-)

eyalroth commented 6 years ago

I guess I have my work cut out for me, don't I? :)

cdrasmussen commented 6 years ago

I'm interested, too. It would be a good project. I'm slow to respond but it will have to be interleaved with everything else.

Is your code where I can see it?

+CDR

On Jan 27, 2018 6:20 PM, "Eyal Roth" notifications@github.com wrote:

I guess I have my work cut out for me, don't I? :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/hoijui/Jawk/issues/11#issuecomment-361023664, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAU9KUdYo-3PSDmOQlttXLra79Ge4Ngks5tO69MgaJpZM4RvRIu .

eyalroth commented 6 years ago

Unfortunately no, most of it is in use on my private repository which I cannot simply publish, but I'll make the effort to transfer it to the public one.

cdrasmussen commented 6 years ago

Instead of existing your while source tree, hope about sending me some patch files? That may be easier for us both.

+CDR

On Jan 28, 2018 2:49 PM, "Eyal Roth" notifications@github.com wrote:

Unfortunately no, most of it is in use on my private repository which I cannot simply publish, but I'll make the effort to transfer it to the public one.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/hoijui/Jawk/issues/11#issuecomment-361090263, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAU9LFm8rQqd8Ud_xHzAivSywJ4TvWnks5tPM9EgaJpZM4RvRIu .

hoijui commented 6 years ago

i have added you as a collaborator ( @eyalroth ). sorry for the non-existent feedback in general. the usual excuse: too much other stuff to do plus, i also do not use JAWK myself anymore since years, and never dived into any of the core code, so reviewing pull requests thoroughly would have taken me a long time. tell me if you need something else; i do get messages per email, and i still read them.