614504 - configure.in has broken link for MDC altbase.h article

Status: RESOLVED DUPLICATE of bug 669829

(Firefox Build System :: General, defect)

Description

[Daniel Holbert [:dholbert]]

configure.in has two copies of this line:
> AC_MSG_ERROR([System header atlbase.h is not available. See org/en/docs/atlbase.h">http://developer.mozilla.org/en/docs/atlbase.h for details on fixing this problem.])

The URL given there is broken, though -- the correct URL is:
  https://developer.mozilla.org/en/atlbase.h

Comment 1

[Daniel Holbert [:dholbert]]

There's a similar problem with another line in configure.in for https://developer.mozilla.org/en-US/docs/oleacc.idl that really wants https://developer.mozilla.org/en/oleacc.idl

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Lets use this bug to patch m-c.

Comment 4

[Austin King [:ozten]]

Attachment: Fixing broken /docs links

Here is a patch against mozilla-central.

Comment 5

[Justin Wood (:Callek)]

Comment on attachment 493095 [details] [diff] [review]
Fixing broken /docs links

I can't officially review m-c code. But it does look good.

Comment 6

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 493095 [details] [diff] [review]
Fixing broken /docs links

Everything looks good except the string changes.  Bringing in Pike to decide whether we can change the strings or whether we need to punt that till after the branch.

Comment 7

[Axel Hecht [:Pike]]

Comment on attachment 493095 [details] [diff] [review]
Fixing broken /docs links

I'm missing a few things here:

- a stable API to link into MDN/MDC. We're struck by this at a few points. Preferably a locale-independent link. Do we have a bug for that?
- the targets of those links are already redirects to https.
- js reference moved

If we're doing such a massive change, we should do it right, IMHO.

Comment 8

[Benjamin Smedberg]

Our source code is English, I don't think we need locale-independent links here.

Comment 9

[Rain [:rain1] (inactive)]

We seem to have gotten rid of the link to atlbase.h a while back.

Comment 10

[Daniel Holbert [:dholbert]]

yup, looks like bug 669829 removed this chunk.

Duping to that bug, since the original issue here was "omg broken link" and bug 669829 fixed that (by removing the link). :)