Hi All,

 I divided the patch which David sent into two.
 1) 001-jboss.patch
    It is the patch which changed '\n' to '\'.
 2) 002-jboss.patch
    It is a patch for the parameter addition of JAVA_OPTS.
    I added export of JAVA_OPTS from a patch of David.

Regards,
Tomo

2011/11/8 Dejan Muhamedagic <[email protected]>:
> Hi Lars,
>
> On Sun, Nov 06, 2011 at 09:56:12PM +0100, Lars Ellenberg wrote:
>> On Fri, Nov 04, 2011 at 09:42:50AM -0500, David Gersic wrote:
>> > >>> On 11/3/2011 at 11:20 AM, Dejan Muhamedagic <[email protected]> wrote:
>> >
>> > > Hunks 2 and 3 fail, don't know if it's due to space being
>> > > mangled or the jboss RA version you worked on is old:
>> >
>> > I started with the newest JBoss RA I could find, but that was a while ago. 
>> > Where can I get the current one?
>> >
>> >
>> > > Also, was there a reason removing \n in the export lines?
>> >
>> > To be honest, I don't remember. I think I had problems getting it working 
>> > with them in. I'll re-check that though.
>>
>> They are wrong.
>> They would try to execute some binary named "n".
>
> Really? Oops. I thought that this was tested and I think that
> there are some that use the RA.
>
>> Yes, please, remove these bogus \n.
>
> No objection, as long as it's in a separate patch :)
>
> Cheers,
>
> Dejan
>
>> --
>> : Lars Ellenberg
>> : LINBIT | Your Way to High Availability
>> : DRBD/HA support and consulting http://www.linbit.com
>> _______________________________________________________
>> Linux-HA-Dev: [email protected]
>> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
>> Home Page: http://linux-ha.org/
> _______________________________________________________
> Linux-HA-Dev: [email protected]
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
>

Attachment: 001-jboss.patch
Description: Binary data

Attachment: 002-jboss.patch
Description: Binary data

_______________________________________________________
Linux-HA-Dev: [email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to