[prev in list] [next in list] [prev in thread] [next in thread] 

List:       pptpclient-devel
Subject:    Re: [pptp-devel] Re: installing with RH 9 rpm's
From:       Paul Moore <pmoore () blueprintconsulting ! com>
Date:       2003-04-25 20:52:37
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi James,

Thanks for the feedback - I've rewritten the whole lot taking into
account your comments.

On Thu, 2003-04-24 at 09:42, James Cameron wrote:

> On Thu, Apr 24, 2003 at 02:23:07AM +1000, Paul Moore wrote:
> > I can get it working by following the instructions outlined at
> > http://pptpclient.sourceforge.net/howto-redhat-90-build.phtml
> 
> Good.
> 
> > I was using "ppp-2.4.2_cvs_20021120-1.i386.rpm" with "./kernelmod.sh"
> > which failed,
> 
> This is expected.  My instructions say use -b.  More info for why at
> http://pptpclient.sourceforge.net/howto-diagnosis.phtml#pppd_options

Amazing what happens when I read the documentation properly.

> > but when using "ppp-2.4.2-b1.i386.rpm" with "./kernelmod.sh -b" it works
> > just fine.
> 
> This is untested by me.  Nice to know it works for you.
> 
> While the kernelmod.sh documentation states that -b requires 2.4.2-b1 or
> later, the CVS version of 20021120 contains the feature that -b needs.
> 
> > See my script at http://www.blueprintconsulting.com/setup-pptpd for all the
> > details.
> 

Now obseleted by http://www.blueprintconsulting.com/pptpsetup 
http://www.blueprintconsulting.com/pptpsetup/pptp-setup 
http://www.blueprintconsulting.com/pptpsetup/pptpdsetup-1.1-1.src.rpm 

Firewalling and IP forwarding isn't included in the new stuff 'though.
 

> Neat.  Here's some contributions.
> 
> Your script claims to be under the GNU GPL but does not clearly mention
> the copyright holder (presumably you).  Add "Copyright (C) 2003 Paul
> Moore".

Fixed.

> The file name on the web server is setup-pptpd, but the file itself
> identifies as setup-pptp.sh.  There's no explanation of what the script
> actually does, other than "setup pptp"; but which - the server or the
> client?  You don't need the client to set up a server, and yet your
> script appears to install both.  I'm confused.

Yes, it's a test script to install both client and server for drop-in
replacements for MS systems.
Right now it mostly focusses on the server, but I'll include
configuration code for the client
and the GUI for the client eventually.

> In configure_optionspptpd() the +mppe-128 option would be better
> expressed in full as require-mppe-128, and might supercede require-mppe
> in my option list higher up.

OK, fixed. Thanks for that. Is require-mppe a superset of
require-mppe-128? Assuming so, 
I'll just disable the require-mppe-128 and hoping the require-mppe will
result in either
mppe-40 or mppe-128 being negotiated.

> You might like to explore the technique of meta-packaging, which is
> creating an RPM file that contains this script of yours as the make
> step.

Not quite sure what metapackaging means, but I think I've either done
this or made this
less necessary by moving all the intelligent stuff into an RPM (Urls
above).

I've tested the scripts on RedHat 9 kernel 2.4.20-9 and RedHat 8 kernel
2.4.18-14.

-- 
Paul Moore <pmoore@blueprintconsulting.com>
Blueprint Consulting Pty Ltd

[Attachment #5 (text/html)]

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/1.1.9">
</HEAD>
<BODY>
Hi James,<BR>
<BR>
Thanks for the feedback - I've rewritten the whole lot taking into account your \
comments.<BR> <BR>
On Thu, 2003-04-24 at 09:42, James Cameron wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>On Thu, Apr 24, 2003 at 02:23:07AM +1000, Paul \
Moore wrote: &gt; I can get it working by following the instructions outlined at
&gt; </FONT><A HREF="http://pptpclient.sourceforge.net/howto-redhat-90-build.phtml"><FONT \
SIZE="3">http://pptpclient.sourceforge.net/howto-redhat-90-build.phtml</FONT></A> \
<FONT COLOR="#737373" SIZE="3"> Good.

&gt; I was using &quot;ppp-2.4.2_cvs_20021120-1.i386.rpm&quot; with \
&quot;./kernelmod.sh&quot; &gt; which failed,

This is expected.  My instructions say use -b.  More info for why at</FONT>
<A HREF="http://pptpclient.sourceforge.net/howto-diagnosis.phtml#pppd_options"><FONT \
SIZE="3">http://pptpclient.sourceforge.net/howto-diagnosis.phtml#pppd_options</I></FONT></A></PRE>
 </BLOCKQUOTE>
<PRE><FONT SIZE="3">Amazing what happens when I read the documentation \
properly.</FONT></PRE> <BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>&gt; but when using \
&quot;ppp-2.4.2-b1.i386.rpm&quot; with &quot;./kernelmod.sh -b&quot; it works &gt; \
just fine.

This is untested by me.  Nice to know it works for you.

While the kernelmod.sh documentation states that -b requires 2.4.2-b1 or
later, the CVS version of 20021120 contains the feature that -b needs.

&gt; See my script at </FONT><A \
HREF="http://www.blueprintconsulting.com/setup-pptpd"><FONT \
SIZE="3">http://www.blueprintconsulting.com/setup-pptpd</FONT></A><FONT \
COLOR="#737373" SIZE="3"> for all the &gt; details.
</I></FONT></PRE>
</BLOCKQUOTE>
<FONT SIZE="3">Now obseleted by </FONT><A \
HREF="http://www.blueprintconsulting.com/pptpsetup"><FONT \
SIZE="3">http://www.blueprintconsulting.com/pptpsetup</FONT></A><FONT SIZE="3"> \
</FONT><BR> <A HREF="http://www.blueprintconsulting.com/pptpsetup/pptpconfig"><FONT \
SIZE="3">http://www.blueprintconsulting.com/pptpsetup/pptp-setup</FONT></A><FONT \
SIZE="3"> </FONT><BR> <A \
HREF="http://www.blueprintconsulting.com/pptpsetup/pptpdsetup-1.1-1.src.rpm"><FONT \
SIZE="3">http://www.blueprintconsulting.com/pptpsetup/pptpdsetup-1.1-1.src.rpm</FONT></A><FONT \
SIZE="3"> <BR> <BR>
Firewalling and IP forwarding isn't included in the new stuff 'though.<BR>
 </FONT>
<BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>Neat.  Here's some contributions.

Your script claims to be under the GNU GPL but does not clearly mention
the copyright holder (presumably you).  Add &quot;Copyright (C) 2003 Paul
Moore&quot;.</I></FONT></PRE>
</BLOCKQUOTE>
<FONT SIZE="3">Fixed.</FONT>
<BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>The file name on the web server is \
setup-pptpd, but the file itself identifies as setup-pptp.sh.  There's no explanation \
of what the script actually does, other than &quot;setup pptp&quot;; but which - the \
server or the client?  You don't need the client to set up a server, and yet your
script appears to install both.&nbsp; I'm confused.</I></FONT></PRE>
</BLOCKQUOTE>
<FONT SIZE="3">Yes, it's a test script to install both client and server for drop-in \
replacements for MS systems.<BR> Right now it mostly focusses on the server, but I'll \
include configuration code for the client<BR> and the GUI for the client \
eventually.</FONT> <BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>In configure_optionspptpd() the +mppe-128 \
option would be better expressed in full as require-mppe-128, and might supercede \
require-mppe in my option list higher up.</I></FONT></PRE>
</BLOCKQUOTE>
<FONT SIZE="3">OK, fixed. Thanks for that. Is require-mppe a superset of \
require-mppe-128? Assuming so, <BR> I'll just disable the require-mppe-128 and hoping \
the require-mppe will result in either<BR> mppe-40 or mppe-128 being \
negotiated.</FONT> <BLOCKQUOTE TYPE=CITE>
<PRE><FONT COLOR="#737373" SIZE="3"><I>You might like to explore the technique of \
meta-packaging, which is creating an RPM file that contains this script of yours as \
the make step.</I></FONT></PRE>
</BLOCKQUOTE>
<FONT SIZE="3">Not quite sure what metapackaging means, but I think I've either done \
this or made this<BR> less necessary by moving all the intelligent stuff into an RPM \
(Urls above).<BR> <BR>
I've tested the scripts on RedHat 9 kernel 2.4.20-9 and RedHat 8 kernel \
2.4.18-</FONT>14. <PRE><TABLE CELLSPACING="0" CELLPADDING="0" WIDTH="100%">
<TR>
<TD>
-- <BR>
Paul Moore &lt;<A HREF="mailto:pmoore@blueprintconsulting.com">pmoore@blueprintconsulting.com</A>&gt;<BR>
 Blueprint Consulting Pty Ltd
</TD>
</TR>
</TABLE>
</PRE>
</BODY>
</HTML>


["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic