Thread: docs cleanup patch

docs cleanup patch

From
Josh Kupershmidt
Date:
Hi all,
Here's a patch which mostly fixes broken URLs in code comments.

Summary of doc. changes:
 * heapfuncs.c: fix awkward comment phrasing

I also tried to fix as many broken URLs as I could find.
 * imath.h, imath.c: homepage for M.J. Fromberger moved
 * sha1.c, sha1.h: new location of FIPS pub 180-1
 * sha2.c: changed URL of PDF describing SHA-256/384/512 to one of the
only active links I could find. archive.org still has the old copy at
http://web.archive.org/web/*/http://csrc.nist.gov/cryptval/shs/sha256-384-512.pdf
 * spell.h: I think this was just pointing to the manpage for
hunspell. I changed to the project page for hunspell, though maybe
it'd be better to point to some other copy of the man page somewhere,
e.g. http://pwet.fr/man/linux/fichiers_speciaux/hunspell
 * dirmod.c: codeproject.com has shuffled pages

Didn't change:
 * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
down, and I can't find an active copy anywhere else, so I didn't try
to change this. archive.org still has it though (
http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
)

Josh

Attachment

Re: docs cleanup patch

From
Magnus Hagander
Date:
On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:
> Hi all,
> Here's a patch which mostly fixes broken URLs in code comments.
>
> Summary of doc. changes:
>  * heapfuncs.c: fix awkward comment phrasing
>
> I also tried to fix as many broken URLs as I could find.
>  * imath.h, imath.c: homepage for M.J. Fromberger moved
>  * sha1.c, sha1.h: new location of FIPS pub 180-1
>  * sha2.c: changed URL of PDF describing SHA-256/384/512 to one of the
> only active links I could find. archive.org still has the old copy at
> http://web.archive.org/web/*/http://csrc.nist.gov/cryptval/shs/sha256-384-512.pdf
>  * spell.h: I think this was just pointing to the manpage for
> hunspell. I changed to the project page for hunspell, though maybe
> it'd be better to point to some other copy of the man page somewhere,
> e.g. http://pwet.fr/man/linux/fichiers_speciaux/hunspell
>  * dirmod.c: codeproject.com has shuffled pages

Thanks, applied!

> Didn't change:
>  * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
> down, and I can't find an active copy anywhere else, so I didn't try
> to change this. archive.org still has it though (
> http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
> )

The page in archive.org also refers to skip.org, which has expired and
been eaten. Perhaps we should just remove that comment completely?


--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

Re: docs cleanup patch

From
Alvaro Herrera
Date:
Magnus Hagander escribió:
> On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:

> > Didn't change:
> >  * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
> > down, and I can't find an active copy anywhere else, so I didn't try
> > to change this. archive.org still has it though (
> > http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
> > )
>
> The page in archive.org also refers to skip.org, which has expired and
> been eaten. Perhaps we should just remove that comment completely?

Eh?  The above link works for me (and contains suggested DH parameters,
though not in the format expected by be-secure.c).

--
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

Re: docs cleanup patch

From
Magnus Hagander
Date:
On Wed, Apr 14, 2010 at 12:15 AM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> Magnus Hagander escribió:
>> On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:
>
>> > Didn't change:
>> >  * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
>> > down, and I can't find an active copy anywhere else, so I didn't try
>> > to change this. archive.org still has it though (
>> > http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
>> > )
>>
>> The page in archive.org also refers to skip.org, which has expired and
>> been eaten. Perhaps we should just remove that comment completely?
>
> Eh?  The above link works for me (and contains suggested DH parameters,
> though not in the format expected by be-secure.c).

Interesting - must be a routing issue or something then, because I
still can't get to it. I've tried from machines in several different
countries, though... Does the content of what you get on the site
actually match what you get on the site itself?


--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

Re: docs cleanup patch

From
Bruce Momjian
Date:
Magnus Hagander wrote:
> On Wed, Apr 14, 2010 at 12:15 AM, Alvaro Herrera
> <alvherre@commandprompt.com> wrote:
> > Magnus Hagander escribi?:
> >> On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:
> >
> >> > Didn't change:
> >> > ?* be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
> >> > down, and I can't find an active copy anywhere else, so I didn't try
> >> > to change this. archive.org still has it though (
> >> > http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
> >> > )
> >>
> >> The page in archive.org also refers to skip.org, which has expired and
> >> been eaten. Perhaps we should just remove that comment completely?
> >
> > Eh? ?The above link works for me (and contains suggested DH parameters,
> > though not in the format expected by be-secure.c).
>
> Interesting - must be a routing issue or something then, because I
> still can't get to it. I've tried from machines in several different
> countries, though... Does the content of what you get on the site
> actually match what you get on the site itself?

I cannot access it.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

Re: docs cleanup patch

From
Alvaro Herrera
Date:
Magnus Hagander escribió:
> On Wed, Apr 14, 2010 at 12:15 AM, Alvaro Herrera
> <alvherre@commandprompt.com> wrote:
> > Magnus Hagander escribió:
> >> On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:
> >
> >> > Didn't change:
> >> >  * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
> >> > down, and I can't find an active copy anywhere else, so I didn't try
> >> > to change this. archive.org still has it though (
> >> > http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
> >> > )
> >>
> >> The page in archive.org also refers to skip.org, which has expired and
> >> been eaten. Perhaps we should just remove that comment completely?
> >
> > Eh?  The above link works for me (and contains suggested DH parameters,
> > though not in the format expected by be-secure.c).
>
> Interesting - must be a routing issue or something then, because I
> still can't get to it. I've tried from machines in several different
> countries, though...

Maybe I was unclear.  The archive.org URL works for me, the skip-vpn.org
one doesn't (neither does skip.org).

I must note that I tried the archive.org URL last night again upon
seeing your email and was very surprised because it told me that that
URL wasn't archived -- in fact none of skip-vpn.org was archived.  Today
it works again.

> Does the content of what you get on the site actually match what you
> get on the site itself?

When you say "the site", which site are you referring to?

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

Re: docs cleanup patch

From
Magnus Hagander
Date:
On Thu, Apr 15, 2010 at 4:48 PM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> Magnus Hagander escribió:
>> On Wed, Apr 14, 2010 at 12:15 AM, Alvaro Herrera
>> <alvherre@commandprompt.com> wrote:
>> > Magnus Hagander escribió:
>> >> On Fri, Mar 26, 2010 at 04:43, Josh Kupershmidt <schmiddy@gmail.com> wrote:
>> >
>> >> > Didn't change:
>> >> >  * be-secure.c: the URL http://www.skip-vpn.org/spec/numbers.html is
>> >> > down, and I can't find an active copy anywhere else, so I didn't try
>> >> > to change this. archive.org still has it though (
>> >> > http://web.archive.org/web/20011212141438/http://www.skip-vpn.org/spec/numbers.html
>> >> > )
>> >>
>> >> The page in archive.org also refers to skip.org, which has expired and
>> >> been eaten. Perhaps we should just remove that comment completely?
>> >
>> > Eh?  The above link works for me (and contains suggested DH parameters,
>> > though not in the format expected by be-secure.c).
>>
>> Interesting - must be a routing issue or something then, because I
>> still can't get to it. I've tried from machines in several different
>> countries, though...
>
> Maybe I was unclear.  The archive.org URL works for me, the skip-vpn.org
> one doesn't (neither does skip.org).

Ok. Then yes, I read you wrong.

I'm unsure if linking to archive.org is a good thing. I guess it's not
likely to go away, but my original comment remains - perhaps we should
just remove it and let people google for the info if they need it.

> I must note that I tried the archive.org URL last night again upon
> seeing your email and was very surprised because it told me that that
> URL wasn't archived -- in fact none of skip-vpn.org was archived.  Today
> it works again.

Strange.


>> Does the content of what you get on the site actually match what you
>> get on the site itself?
>
> When you say "the site", which site are you referring to?

skip-vpn.org.


--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

Re: docs cleanup patch

From
Alvaro Herrera
Date:
Magnus Hagander escribió:

> I'm unsure if linking to archive.org is a good thing. I guess it's not
> likely to go away, but my original comment remains - perhaps we should
> just remove it and let people google for the info if they need it.

I guess the question is: how valuable is it to change the hardcoded
fallback DH parameters?  Surely if someone wants to use different DH
params, they can set them in the documented files.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

Re: docs cleanup patch

From
Magnus Hagander
Date:
On Thu, Apr 15, 2010 at 5:43 PM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> Magnus Hagander escribió:
>
>> I'm unsure if linking to archive.org is a good thing. I guess it's not
>> likely to go away, but my original comment remains - perhaps we should
>> just remove it and let people google for the info if they need it.
>
> I guess the question is: how valuable is it to change the hardcoded
> fallback DH parameters?  Surely if someone wants to use different DH
> params, they can set them in the documented files.

I've never heard anybody ask for that capability. I think that's more
"developer notes" that ended up in comments...

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/