Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch links to https #1589

Closed
wants to merge 4 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Documentation/CodingGuidelines
Original file line number Diff line number Diff line change
Expand Up @@ -518,7 +518,7 @@ For Perl programs:

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Eric Sunshine wrote (reply to this):

On Fri, Sep 22, 2023 at 10:14 AM Josh Soref via GitGitGadget
<gitgitgadget@gmail.com> wrote:
> doc: update links to current pages
>
> Signed-off-by: Josh Soref <jsoref@gmail.com>
> ---
> diff --git a/json-writer.h b/json-writer.h
> @@ -4,7 +4,7 @@
>  /*
>   * JSON data structures are defined at:
>   * [1] https://www.ietf.org/rfc/rfc7159.txt
> - * [2] http://json.org/
> + * [2] https://www.json.org/

Not sure why this change is needed considering that the simpler
s/http/https/ seems sufficient.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Josh Soref wrote (reply to this):

Eric Sunshine wrote:
> > doc: update links to current pages
> > @@ -4,7 +4,7 @@
> > - * [2] http://json.org/
> > + * [2] https://www.json.org/
>
> Not sure why this change is needed considering that the simpler
> s/http/https/ seems sufficient.

This is because that's their preference:

```sh
% curl -s http://json.org/|grep REFRESH
<meta HTTP-EQUIV="REFRESH" content="0;
url=https://www.JSON.org/json-en.html"></head>
```

It's somewhat traditional to respect sites' self-identification.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Eric Sunshine wrote (reply to this):

On Fri, Sep 22, 2023 at 5:31 PM Josh Soref <jsoref@gmail.com> wrote:
> Eric Sunshine wrote:
> > > - * [2] http://json.org/
> > > + * [2] https://www.json.org/
> >
> > Not sure why this change is needed considering that the simpler
> > s/http/https/ seems sufficient.
>
> This is because that's their preference:
>
> ```sh
> % curl -s http://json.org/|grep REFRESH
> <meta HTTP-EQUIV="REFRESH" content="0;
> url=https://www.JSON.org/json-en.html"></head>
> ```
>
> It's somewhat traditional to respect sites' self-identification.

It might be worth calling that out in the commit message of both
patches if you happen to reroll so other reviewers understand the
motivation. (That said, on its own, it's probably not worth a reroll.)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Junio C Hamano wrote (reply to this):

"Josh Soref via GitGitGadget" <gitgitgadget@gmail.com> writes:

> From: Josh Soref <jsoref@gmail.com>
>
> Signed-off-by: Josh Soref <jsoref@gmail.com>
> ---
>  Documentation/CodingGuidelines     | 2 +-
>  Documentation/RelNotes/1.6.2.txt   | 2 +-
>  Documentation/RelNotes/1.6.3.txt   | 2 +-
>  Documentation/RelNotes/1.6.4.txt   | 2 +-
>  Documentation/RelNotes/1.6.5.txt   | 2 +-
>  Documentation/RelNotes/1.6.6.txt   | 2 +-
>  Documentation/git-cvsimport.txt    | 2 +-
>  Documentation/git-format-patch.txt | 2 +-
>  Documentation/git-ls-remote.txt    | 4 ++--
>  Documentation/git.txt              | 2 +-
>  Documentation/gitcore-tutorial.txt | 4 ++--
>  compat/nedmalloc/malloc.c.h        | 2 +-
>  contrib/persistent-https/README    | 2 +-
>  git-gui/git-gui.sh                 | 2 +-
>  gitk-git/gitk                      | 2 +-
>  gitweb/static/js/lib/common-lib.js | 2 +-
>  http.c                             | 2 +-
>  imap-send.c                        | 2 +-
>  json-writer.h                      | 2 +-
>  19 files changed, 21 insertions(+), 21 deletions(-)

Broken links are annoying while reading documentation.  Thank you
for doing the necessary research.

I am not sure if direct replacement to Release Notes (historical
document) is the best way to present these updates, but let's assume
it is OK for now.

> diff --git a/Documentation/git-ls-remote.txt b/Documentation/git-ls-remote.txt
> index 1c4f696ab57..b9c8478a62a 100644
> --- a/Documentation/git-ls-remote.txt
> +++ b/Documentation/git-ls-remote.txt
> @@ -128,7 +128,7 @@ d4ca2e3147b409459955613c152220f4db848ee1	refs/tags/v2.40.0
>  * List all references matching given patterns:
>  +
>  ----
> -$ git ls-remote http://www.kernel.org/pub/scm/git/git.git master seen rc
> +$ git ls-remote https://git.kernel.org/pub/scm/git/git.git master seen rc
>  5fe978a5381f1fbad26a80e682ddd2a401966740	refs/heads/master
>  c781a84b5204fb294c9ccc79f8b3baceeb32c061	refs/heads/seen
>  ----

I am not sure if this change is warranted.  The old URL still works
just fine, even though there are no refs that match "rc" and the
"master" branch is no longer at 5fe978a5, but these are not problem
caused by using the http:// address.  The point of the example is
not about preferred use of HTTP:// over HTTPS://, so this again
falls into "if we were writing this anew, we may have used the other
address, but the original still works, so it is not worth the patch
noise" category, I would think.

> diff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt
> index 2122aeb9769..924a9a97807 100644
> --- a/Documentation/gitcore-tutorial.txt
> +++ b/Documentation/gitcore-tutorial.txt
> @@ -1101,8 +1101,8 @@ Examples.
>  
>  the above are equivalent to:
>  
> -. `git pull http://www.kernel.org/pub/scm/git/git.git/ HEAD`
> -. `git pull http://www.kernel.org/pub/scm/git/git.git/ tag v0.99.1`
> +. `git pull https://git.kernel.org/pub/scm/git/git.git/ HEAD`
> +. `git pull https://git.kernel.org/pub/scm/git/git.git/ tag v0.99.1`

Likewise.

> diff --git a/Documentation/git.txt b/Documentation/git.txt
> index 11228956cd5..c7292eb25d0 100644
> --- a/Documentation/git.txt
> +++ b/Documentation/git.txt
> @@ -1061,7 +1061,7 @@ Authors
>  -------
>  Git was started by Linus Torvalds, and is currently maintained by Junio
>  C Hamano. Numerous contributions have come from the Git mailing list
> -<git@vger.kernel.org>.  http://www.openhub.net/p/git/contributors/summary
> +<git@vger.kernel.org>.  https://openhub.net/p/git/contributors/summary
>  gives you a more complete list of contributors.

OK, even though the former seems to redirect to the latter, so in
that sense it is still "current".

> diff --git a/compat/nedmalloc/malloc.c.h b/compat/nedmalloc/malloc.c.h
> index 5c5357a379f..4b711c6b9ca 100644
> --- a/compat/nedmalloc/malloc.c.h
> +++ b/compat/nedmalloc/malloc.c.h
> @@ -1359,7 +1359,7 @@ LONG __cdecl _InterlockedExchange(LONG volatile *Target, LONG Value);
>    /* --[ start GCC compatibility ]----------------------------------------------
>     * Compatibility <intrin_x86.h> header for GCC -- GCC equivalents of intrinsic
>     * Microsoft Visual C++ functions. Originally developed for the ReactOS
> -   * (<http://www.reactos.org/>) and TinyKrnl (<http://www.tinykrnl.org/>)
> +   * (<https://reactos.org/>) and TinyKrnl (<http://www.tinykrnl.org/>)

Likewise for reactos; the tinykrnl one does not seem to be current.

In any case, this is borrowed code, so I'd rather not touch it and
risk giving a wrong page that is not a replacement.

> diff --git a/contrib/persistent-https/README b/contrib/persistent-https/README
> index 2ad95893c27..2c9bec91066 100644
> --- a/contrib/persistent-https/README
> +++ b/contrib/persistent-https/README
> @@ -60,7 +60,7 @@ https://kernel.googlesource.com/pub/scm/git/git
>  
>  PREREQUISITES
>  
> -The code is written in Go (http://golang.org/) and the Go compiler is
> +The code is written in Go (https://go.dev/) and the Go compiler is

Likewise.

>  required. Currently, the compiler must be built and installed from tip
>  of source, in order to include a fix in the reverse http proxy:
>  https://code.google.com/p/go/source/detail?r=a615b796570a2cd8591884767a7d67ede74f6648

This is an interesting one.  code.google.com is no longer tehre

> diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh
> index 1d6102e0674..4a11d590b68 100755
> --- a/git-gui/git-gui.sh
> +++ b/git-gui/git-gui.sh
> @@ -2390,7 +2390,7 @@ proc do_quit {{rc {1}}} {
>  	set ret_code $rc
>  
>  	# Briefly enable send again, working around Tk bug
> -	# http://sourceforge.net/tracker/?func=detail&atid=112997&aid=1821174&group_id=12997
> +	# https://sourceforge.net/p/tktoolkit/bugs/2343/
>  	tk appname [appname]
>  
>  	destroy .
> diff --git a/gitk-git/gitk b/gitk-git/gitk
> index 1db46977df0..7a087f123d7 100755
> --- a/gitk-git/gitk
> +++ b/gitk-git/gitk
> @@ -12472,7 +12472,7 @@ if {[tk windowingsystem] eq "aqua"} {
>  
>  catch {
>      # follow the XDG base directory specification by default. See
> -    # http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
> +    # https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html
>      if {[info exists env(XDG_CONFIG_HOME)] && $env(XDG_CONFIG_HOME) ne ""} {
>          # XDG_CONFIG_HOME environment variable is set
>          set config_file [file join $env(XDG_CONFIG_HOME) git gitk]
> diff --git a/gitweb/static/js/lib/common-lib.js b/gitweb/static/js/lib/common-lib.js
> index 17b1796496d..99e3eb8c3d9 100644
> --- a/gitweb/static/js/lib/common-lib.js
> +++ b/gitweb/static/js/lib/common-lib.js
> @@ -137,7 +137,7 @@ function addCssRule(selector, style) {
>   *   http://www.dustindiaz.com/getelementsbyclass/
>   *   https://stackoverflow.com/questions/1818865/do-we-have-getelementsbyclassname-in-javascript
>   *
> - * See also http://ejohn.org/blog/getelementsbyclassname-speed-comparison/
> + * See also https://johnresig.com/blog/getelementsbyclassname-speed-comparison/
>   *
>   * @param {String} class: name of _single_ class to find
>   * @param {String} [taghint] limit search to given tags
> diff --git a/http.c b/http.c
> index e138b4b96fb..33c6011e8d8 100644
> --- a/http.c
> +++ b/http.c
> @@ -1877,7 +1877,7 @@ static void write_accept_language(struct strbuf *buf)
>  	 * MAX_DECIMAL_PLACES must not be larger than 3. If it is larger than
>  	 * that, q-value will be smaller than 0.001, the minimum q-value the
>  	 * HTTP specification allows. See
> -	 * http://tools.ietf.org/html/rfc7231#section-5.3.1 for q-value.
> +	 * https://datatracker.ietf.org/doc/html/rfc7231#section-5.3.1 for q-value.
>  	 */
>  	const int MAX_DECIMAL_PLACES = 3;
>  	const int MAX_LANGUAGE_TAGS = 1000;
> diff --git a/imap-send.c b/imap-send.c
> index 3b2077e39b2..448ca64c052 100644
> --- a/imap-send.c
> +++ b/imap-send.c
> @@ -860,7 +860,7 @@ static void imap_close_store(struct imap_store *ctx)
>  
>  /*
>   * hexchar() and cram() functions are based on the code from the isync
> - * project (http://isync.sf.net/).
> + * project (https://isync.sourceforge.io/).
>   */
>  static char hexchar(unsigned int b)
>  {
> diff --git a/json-writer.h b/json-writer.h
> index de140e54c98..04413bd1afd 100644
> --- a/json-writer.h
> +++ b/json-writer.h
> @@ -4,7 +4,7 @@
>  /*
>   * JSON data structures are defined at:
>   * [1] https://www.ietf.org/rfc/rfc7159.txt
> - * [2] http://json.org/
> + * [2] https://www.json.org/
>   *
>   * The JSON-writer API allows one to build JSON data structures using a
>   * simple wrapper around a "struct strbuf" buffer.  It is intended as a

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Junio C Hamano wrote (reply to this):

"Josh Soref via GitGitGadget" <gitgitgadget@gmail.com> writes:

> From: Josh Soref <jsoref@gmail.com>
>
> Signed-off-by: Josh Soref <jsoref@gmail.com>
> ---
>  Documentation/CodingGuidelines     | 2 +-
>  Documentation/RelNotes/1.6.2.txt   | 2 +-
>  Documentation/RelNotes/1.6.3.txt   | 2 +-
>  Documentation/RelNotes/1.6.4.txt   | 2 +-
>  Documentation/RelNotes/1.6.5.txt   | 2 +-
>  Documentation/RelNotes/1.6.6.txt   | 2 +-
>  Documentation/git-cvsimport.txt    | 2 +-
>  Documentation/git-format-patch.txt | 2 +-
>  Documentation/git-ls-remote.txt    | 4 ++--
>  Documentation/git.txt              | 2 +-
>  Documentation/gitcore-tutorial.txt | 4 ++--
>  compat/nedmalloc/malloc.c.h        | 2 +-
>  contrib/persistent-https/README    | 2 +-
>  git-gui/git-gui.sh                 | 2 +-
>  gitk-git/gitk                      | 2 +-
>  gitweb/static/js/lib/common-lib.js | 2 +-
>  http.c                             | 2 +-
>  imap-send.c                        | 2 +-
>  json-writer.h                      | 2 +-
>  19 files changed, 21 insertions(+), 21 deletions(-)

Broken links are annoying while reading documentation.  Thank you
for doing the necessary research.

I am not sure if direct replacement to Release Notes (historical
document) is the best way to present these updates, but let's assume
it is OK for now.

> diff --git a/Documentation/git-ls-remote.txt b/Documentation/git-ls-remote.txt
> index 1c4f696ab57..b9c8478a62a 100644
> --- a/Documentation/git-ls-remote.txt
> +++ b/Documentation/git-ls-remote.txt
> @@ -128,7 +128,7 @@ d4ca2e3147b409459955613c152220f4db848ee1	refs/tags/v2.40.0
>  * List all references matching given patterns:
>  +
>  ----
> -$ git ls-remote http://www.kernel.org/pub/scm/git/git.git master seen rc
> +$ git ls-remote https://git.kernel.org/pub/scm/git/git.git master seen rc
>  5fe978a5381f1fbad26a80e682ddd2a401966740	refs/heads/master
>  c781a84b5204fb294c9ccc79f8b3baceeb32c061	refs/heads/seen
>  ----

I am not sure if this change is warranted.  The old URL still works
just fine, even though there are no refs that match "rc" and the
"master" branch is no longer at 5fe978a5, but these are not problem
caused by using the http:// address.  The point of the example is
not about preferred use of HTTP:// over HTTPS://, so this again
falls into "if we were writing this anew, we may have used the other
address, but the original still works, so it is not worth the patch
noise" category, I would think.

> diff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt
> index 2122aeb9769..924a9a97807 100644
> --- a/Documentation/gitcore-tutorial.txt
> +++ b/Documentation/gitcore-tutorial.txt
> @@ -1101,8 +1101,8 @@ Examples.
>  
>  the above are equivalent to:
>  
> -. `git pull http://www.kernel.org/pub/scm/git/git.git/ HEAD`
> -. `git pull http://www.kernel.org/pub/scm/git/git.git/ tag v0.99.1`
> +. `git pull https://git.kernel.org/pub/scm/git/git.git/ HEAD`
> +. `git pull https://git.kernel.org/pub/scm/git/git.git/ tag v0.99.1`

Likewise.

> diff --git a/Documentation/git.txt b/Documentation/git.txt
> index 11228956cd5..c7292eb25d0 100644
> --- a/Documentation/git.txt
> +++ b/Documentation/git.txt
> @@ -1061,7 +1061,7 @@ Authors
>  -------
>  Git was started by Linus Torvalds, and is currently maintained by Junio
>  C Hamano. Numerous contributions have come from the Git mailing list
> -<git@vger.kernel.org>.  http://www.openhub.net/p/git/contributors/summary
> +<git@vger.kernel.org>.  https://openhub.net/p/git/contributors/summary
>  gives you a more complete list of contributors.

OK, even though the former seems to redirect to the latter, so in
that sense it is still "current".

> diff --git a/compat/nedmalloc/malloc.c.h b/compat/nedmalloc/malloc.c.h
> index 5c5357a379f..4b711c6b9ca 100644
> --- a/compat/nedmalloc/malloc.c.h
> +++ b/compat/nedmalloc/malloc.c.h
> @@ -1359,7 +1359,7 @@ LONG __cdecl _InterlockedExchange(LONG volatile *Target, LONG Value);
>    /* --[ start GCC compatibility ]----------------------------------------------
>     * Compatibility <intrin_x86.h> header for GCC -- GCC equivalents of intrinsic
>     * Microsoft Visual C++ functions. Originally developed for the ReactOS
> -   * (<http://www.reactos.org/>) and TinyKrnl (<http://www.tinykrnl.org/>)
> +   * (<https://reactos.org/>) and TinyKrnl (<http://www.tinykrnl.org/>)

Likewise for reactos; the tinykrnl one does not seem to be current.

In any case, this is borrowed code, so I'd rather not touch it and
risk giving a wrong page that is not a replacement.

> diff --git a/contrib/persistent-https/README b/contrib/persistent-https/README
> index 2ad95893c27..2c9bec91066 100644
> --- a/contrib/persistent-https/README
> +++ b/contrib/persistent-https/README
> @@ -60,7 +60,7 @@ https://kernel.googlesource.com/pub/scm/git/git
>  
>  PREREQUISITES
>  
> -The code is written in Go (http://golang.org/) and the Go compiler is
> +The code is written in Go (https://go.dev/) and the Go compiler is

Likewise.

>  required. Currently, the compiler must be built and installed from tip
>  of source, in order to include a fix in the reverse http proxy:
>  https://code.google.com/p/go/source/detail?r=a615b796570a2cd8591884767a7d67ede74f6648

This is an interesting one.  code.google.com is no longer there and
the updated URL seems to be at https://github.com/golang/go somewhere.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Elijah Newren wrote (reply to this):

On Sun, Sep 24, 2023 at 8:06 AM Josh Soref via GitGitGadget
<gitgitgadget@gmail.com> wrote:
>
> From: Josh Soref <jsoref@gmail.com>
>
> It's somewhat traditional to respect sites' self-identification.

I don't understand this comment; was it meant for patch 1?

[...]
> --- a/json-writer.h
> +++ b/json-writer.h
> @@ -4,7 +4,7 @@
>  /*
>   * JSON data structures are defined at:
>   * [1] https://www.ietf.org/rfc/rfc7159.txt
> - * [2] http://json.org/
> + * [2] https://www.json.org/
>   *
>   * The JSON-writer API allows one to build JSON data structures using a
>   * simple wrapper around a "struct strbuf" buffer.  It is intended as a

Ah, you did fix the http/https thing for json, you just moved it to
patch 2 because you also added the 'www.'.  Got it.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Git mailing list, Josh Soref wrote (reply to this):

Elijah Newren wrote:
> > It's somewhat traditional to respect sites' self-identification.
>
> I don't understand this comment; was it meant for patch 1?

I'm trying to say that these sites tend to redirect to these forms as
opposed to the previous forms.

It applies to both commits.

The reason to retain two distinct commits is that ideally the https
only commit should be easier to review as it's generally just adding
an `s` (in one case I've added a trailing `/` before a `.` that was
used to end a sentence and which would result in a broken link if a
user clicked on it...).

> Ah, you did fix the http/https thing for json, you just moved it to
> patch 2 because you also added the 'www.'.  Got it.

This confused too many people. I'm going to swap the order.

For Python scripts:

- We follow PEP-8 (http://www.python.org/dev/peps/pep-0008/).
- We follow PEP-8 (https://peps.python.org/pep-0008/).

- As a minimum, we aim to be compatible with Python 2.7.

Expand Down
2 changes: 1 addition & 1 deletion Documentation/MyFirstContribution.txt
Original file line number Diff line number Diff line change
Expand Up @@ -833,7 +833,7 @@ Johannes Schindelin to make life as a Git contributor easier for those used to
the GitHub PR workflow. It allows contributors to open pull requests against its
mirror of the Git project, and does some magic to turn the PR into a set of
emails and send them out for you. It also runs the Git continuous integration
suite for you. It's documented at http://gitgitgadget.github.io.
suite for you. It's documented at https://gitgitgadget.github.io/.

[[create-fork]]
=== Forking `git/git` on GitHub
Expand Down
2 changes: 1 addition & 1 deletion Documentation/RelNotes/1.6.2.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ To ease the transition plan, the receiving repository of such a
push running this release will issue a big warning when the
configuration variable is missing. Please refer to:

http://git.or.cz/gitwiki/GitFaq#non-bare
https://archive.kernel.org/oldwiki/git.wiki.kernel.org/index.php/GitFaq.html#non-bare
https://lore.kernel.org/git/7vbptlsuyv.fsf@gitster.siamese.dyndns.org/

for more details on the reason why this change is needed and the
Expand Down
2 changes: 1 addition & 1 deletion Documentation/RelNotes/1.6.3.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ To ease the transition plan, the receiving repository of such a
push running this release will issue a big warning when the
configuration variable is missing. Please refer to:

http://git.or.cz/gitwiki/GitFaq#non-bare
https://archive.kernel.org/oldwiki/git.wiki.kernel.org/index.php/GitFaq.html#non-bare
https://lore.kernel.org/git/7vbptlsuyv.fsf@gitster.siamese.dyndns.org/

for more details on the reason why this change is needed and the
Expand Down
2 changes: 1 addition & 1 deletion Documentation/RelNotes/1.6.4.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ To ease the transition plan, the receiving repository of such a
push running this release will issue a big warning when the
configuration variable is missing. Please refer to:

http://git.or.cz/gitwiki/GitFaq#non-bare
https://archive.kernel.org/oldwiki/git.wiki.kernel.org/index.php/GitFaq.html#non-bare
https://lore.kernel.org/git/7vbptlsuyv.fsf@gitster.siamese.dyndns.org/

for more details on the reason why this change is needed and the
Expand Down
2 changes: 1 addition & 1 deletion Documentation/RelNotes/1.6.5.txt
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ To ease the transition plan, the receiving repository of such a
push running this release will issue a big warning when the
configuration variable is missing. Please refer to:

http://git.or.cz/gitwiki/GitFaq#non-bare
https://archive.kernel.org/oldwiki/git.wiki.kernel.org/index.php/GitFaq.html#non-bare
https://lore.kernel.org/git/7vbptlsuyv.fsf@gitster.siamese.dyndns.org/

for more details on the reason why this change is needed and the
Expand Down
2 changes: 1 addition & 1 deletion Documentation/RelNotes/1.6.6.txt
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ users will fare this time.

Please refer to:

http://git.or.cz/gitwiki/GitFaq#non-bare
https://archive.kernel.org/oldwiki/git.wiki.kernel.org/index.php/GitFaq.html#non-bare
https://lore.kernel.org/git/7vbptlsuyv.fsf@gitster.siamese.dyndns.org/

for more details on the reason why this change is needed and the
Expand Down
4 changes: 2 additions & 2 deletions Documentation/git-cvsimport.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ DESCRIPTION
deprecated; it does not work with cvsps version 3 and later. If you are
performing a one-shot import of a CVS repository consider using
http://cvs2svn.tigris.org/cvs2git.html[cvs2git] or
http://www.catb.org/esr/cvs-fast-export/[cvs-fast-export].
https://gitlab.com/esr/cvs-fast-export[cvs-fast-export].

Imports a CVS repository into Git. It will either create a new
repository, or incrementally import into an existing one.
Expand Down Expand Up @@ -221,7 +221,7 @@ Problems related to tags:
If you suspect that any of these issues may apply to the repository you
want to import, consider using cvs2git:

* cvs2git (part of cvs2svn), `http://subversion.apache.org/`
* cvs2git (part of cvs2svn), `https://subversion.apache.org/`

GIT
---
Expand Down
4 changes: 2 additions & 2 deletions Documentation/git-format-patch.txt
Original file line number Diff line number Diff line change
Expand Up @@ -610,8 +610,8 @@ Approach #3 (external editor)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The following Thunderbird extensions are needed:
AboutConfig from http://aboutconfig.mozdev.org/ and
External Editor from http://globs.org/articles.php?lng=en&pg=8
AboutConfig from https://mjg.github.io/AboutConfig/ and
External Editor from https://globs.org/articles.php?lng=en&pg=8

1. Prepare the patch as a text file using your method of choice.

Expand Down
2 changes: 1 addition & 1 deletion Documentation/git-imap-send.txt
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ flames ridiculing you if you don't check this.

Thunderbird in particular is known to be problematic. Thunderbird
users may wish to visit this web page for more information:
http://kb.mozillazine.org/Plain_text_e-mail_-_Thunderbird#Completely_plain_email
https://kb.mozillazine.org/Plain_text_e-mail_-_Thunderbird#Completely_plain_email

SEE ALSO
--------
Expand Down
2 changes: 1 addition & 1 deletion Documentation/git-send-email.txt
Original file line number Diff line number Diff line change
Expand Up @@ -454,7 +454,7 @@ have been specified, in which case default to 'compose'.
998 characters unless a suitable transfer encoding
('auto', 'base64', or 'quoted-printable') is used;
this is due to SMTP limits as described by
http://www.ietf.org/rfc/rfc5322.txt.
https://www.ietf.org/rfc/rfc5322.txt.
--
+
Default is the value of `sendemail.validate`; if this is not set,
Expand Down
2 changes: 1 addition & 1 deletion Documentation/git.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1071,7 +1071,7 @@ Authors
-------
Git was started by Linus Torvalds, and is currently maintained by Junio
C Hamano. Numerous contributions have come from the Git mailing list
<git@vger.kernel.org>. http://www.openhub.net/p/git/contributors/summary
<git@vger.kernel.org>. https://openhub.net/p/git/contributors/summary
gives you a more complete list of contributors.

If you have a clone of git.git itself, the
Expand Down
2 changes: 1 addition & 1 deletion Documentation/gitcore-tutorial.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1089,7 +1089,7 @@ the remote repository URL in the local repository's config file
like this:

------------------------------------------------
$ git config remote.linus.url http://www.kernel.org/pub/scm/git/git.git/
$ git config remote.linus.url https://git.kernel.org/pub/scm/git/git.git/
------------------------------------------------

and use the "linus" keyword with 'git pull' instead of the full URL.
Expand Down
4 changes: 2 additions & 2 deletions Documentation/gitprotocol-http.txt
Original file line number Diff line number Diff line change
Expand Up @@ -529,8 +529,8 @@ TODO: Document this further.
REFERENCES
----------

http://www.ietf.org/rfc/rfc1738.txt[RFC 1738: Uniform Resource Locators (URL)]
http://www.ietf.org/rfc/rfc2616.txt[RFC 2616: Hypertext Transfer Protocol -- HTTP/1.1]
https://www.ietf.org/rfc/rfc1738.txt[RFC 1738: Uniform Resource Locators (URL)]
https://www.ietf.org/rfc/rfc2616.txt[RFC 2616: Hypertext Transfer Protocol -- HTTP/1.1]

SEE ALSO
--------
Expand Down
4 changes: 2 additions & 2 deletions Documentation/gitweb.conf.txt
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ $mimetypes_file::

$highlight_bin::
Path to the highlight executable to use (it must be the one from
http://www.andre-simon.de[] due to assumptions about parameters and output).
http://andre-simon.de/zip/download.php[] due to assumptions about parameters and output).
By default set to 'highlight'; set it to full path to highlight
executable if it is not installed on your web server's PATH.
Note that 'highlight' feature must be set for gitweb to actually
Expand Down Expand Up @@ -820,7 +820,7 @@ filesystem (i.e. "$projectroot/$project"), `%h` to the current hash
(\'h' gitweb parameter) and `%b` to the current hash base
(\'hb' gitweb parameter); `%%` expands to \'%'.
+
For example, at the time this page was written, the http://repo.or.cz[]
For example, at the time this page was written, the https://repo.or.cz[]
Git hosting site set it to the following to enable graphical log
(using the third party tool *git-browser*):
+
Expand Down
2 changes: 1 addition & 1 deletion Documentation/gitweb.txt
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ Gitweb provides a web interface to Git repositories. Its features include:
revisions one at a time, viewing the history of the repository.
* Finding commits whose commit messages match a given search term.

See http://repo.or.cz/w/git.git/tree/HEAD:/gitweb/[] for gitweb source code,
See https://repo.or.cz/w/git.git/tree/HEAD:/gitweb/[] for gitweb source code,
browsed using gitweb itself.


Expand Down
2 changes: 1 addition & 1 deletion Documentation/howto/keep-canonical-history-correct.txt
Original file line number Diff line number Diff line change
Expand Up @@ -213,4 +213,4 @@ The procedure will result in a history that looks like this:
B0--B1---------B2
------------

See also http://git-blame.blogspot.com/2013/09/fun-with-first-parent-history.html
See also https://git-blame.blogspot.com/2013/09/fun-with-first-parent-history.html
2 changes: 1 addition & 1 deletion Documentation/signoff-option.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ endif::git-commit[]
the committer has the rights to submit the work under the
project's license or agrees to some contributor representation,
such as a Developer Certificate of Origin.
(See http://developercertificate.org for the one used by the
(See https://developercertificate.org for the one used by the
Linux kernel and Git projects.) Consult the documentation or
leadership of the project to which you're contributing to
understand how the signoffs are used in that project.
Expand Down
2 changes: 1 addition & 1 deletion INSTALL
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ Issues of note:
interacting with svn repositories with "git svn"). If you can
live without these, use NO_PERL. Note that recent releases of
Redhat/Fedora are reported to ship Perl binary package with some
core modules stripped away (see http://lwn.net/Articles/477234/),
core modules stripped away (see https://lwn.net/Articles/477234/),
so you might need to install additional packages other than Perl
itself, e.g. Digest::MD5, File::Spec, File::Temp, Net::Domain,
Net::SMTP, and Time::HiRes.
Expand Down
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ include shared.mak
# Define NO_DEFLATE_BOUND if your zlib does not have deflateBound.
#
# Define NO_NORETURN if using buggy versions of gcc 4.6+ and profile feedback,
# as the compiler can crash (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49299)
# as the compiler can crash (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49299)
#
# Define USE_NSEC below if you want git to care about sub-second file mtimes
# and ctimes. Note that you need recent glibc (at least 2.2.4) for this. On
Expand Down Expand Up @@ -2723,7 +2723,7 @@ $(OBJECTS): %.o: %.c GIT-CFLAGS $(missing_dep_dirs) $(missing_compdb_dir)

ifdef USE_COMPUTED_HEADER_DEPENDENCIES
# Take advantage of gcc's on-the-fly dependency generation
# See <http://gcc.gnu.org/gcc-3.0/features.html>.
# See <https://gcc.gnu.org/gcc-3.0/features.html>.
dep_files_present := $(wildcard $(dep_files))
ifneq ($(dep_files_present),)
include $(dep_files_present)
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ string translations (localization l10) should see [po/README.md][]
To subscribe to the list, send an email with just "subscribe git" in
the body to majordomo@vger.kernel.org (not the Git list). The mailing
list archives are available at <https://lore.kernel.org/git/>,
<http://marc.info/?l=git> and other archival sites.
<https://marc.info/?l=git> and other archival sites.

Issues which are security relevant should be disclosed privately to
the Git Security mailing list <git-security@googlegroups.com>.
Expand Down
2 changes: 1 addition & 1 deletion git-cvsimport.perl
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ sub conn {
# Use a HTTP Proxy. Only works for HTTP proxies that
# don't require user authentication
#
# See: http://www.ietf.org/rfc/rfc2817.txt
# See: https://www.ietf.org/rfc/rfc2817.txt

$s = IO::Socket::INET->new(PeerHost => $proxyhost, PeerPort => $proxyport);
die "Socket to $proxyhost: $!\n" unless defined $s;
Expand Down
6 changes: 3 additions & 3 deletions git-gui/git-gui.sh
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with this program; if not, see <http://www.gnu.org/licenses/>.}]
along with this program; if not, see <https://www.gnu.org/licenses/>.}]

######################################################################
##
Expand Down Expand Up @@ -2367,7 +2367,7 @@ proc do_quit {{rc {1}}} {
set ret_code $rc

# Briefly enable send again, working around Tk bug
# http://sourceforge.net/tracker/?func=detail&atid=112997&aid=1821174&group_id=12997
# https://sourceforge.net/p/tktoolkit/bugs/2343/
tk appname [appname]

destroy .
Expand Down Expand Up @@ -3052,7 +3052,7 @@ if {$doc_path ne {}} {
if {[file isfile $doc_path]} {
set doc_url "file:$doc_path"
} else {
set doc_url {http://www.kernel.org/pub/software/scm/git/docs/}
set doc_url {https://www.kernel.org/pub/software/scm/git/docs/}
}

proc start_browser {url} {
Expand Down
2 changes: 1 addition & 1 deletion git-gui/lib/encoding.tcl
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# (Copied from gitk, commit fd8ccbec4f0161)

# This list of encoding names and aliases is distilled from
# http://www.iana.org/assignments/character-sets.
# https://www.iana.org/assignments/character-sets.
# Not all of them are supported by Tcl.
set encoding_aliases {
{ ANSI_X3.4-1968 iso-ir-6 ANSI_X3.4-1986 ISO_646.irv:1991 ASCII
Expand Down
2 changes: 1 addition & 1 deletion git-gui/po/README
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ in your language?
If you do not know what your language should be named, you need to find
it. This currently follows ISO 639-1 two letter codes:

http://www.loc.gov/standards/iso639-2/php/code_list.php
https://www.loc.gov/standards/iso639-2/php/code_list.php

For example, if you are preparing a translation for Afrikaans, the
language code is "af". If there already is a translation for your
Expand Down
4 changes: 2 additions & 2 deletions git-instaweb.sh
Original file line number Diff line number Diff line change
Expand Up @@ -432,7 +432,7 @@ mongoose_conf() {
# Mongoose web server configuration file.
# Lines starting with '#' and empty lines are ignored.
# For detailed description of every option, visit
# http://code.google.com/p/mongoose/wiki/MongooseManual
# https://code.google.com/p/mongoose/wiki/MongooseManual

root $root
ports $port
Expand All @@ -458,7 +458,7 @@ plackup_conf () {
#!$PERL

# gitweb - simple web interface to track changes in git repositories
# PSGI wrapper and server starter (see http://plackperl.org)
# PSGI wrapper and server starter (see https://plackperl.org)

use strict;

Expand Down
4 changes: 2 additions & 2 deletions gitk-git/gitk
Original file line number Diff line number Diff line change
Expand Up @@ -11956,7 +11956,7 @@ proc formatdate {d} {
}

# This list of encoding names and aliases is distilled from
# http://www.iana.org/assignments/character-sets.
# https://www.iana.org/assignments/character-sets.
# Not all of them are supported by Tcl.
set encoding_aliases {
{ ANSI_X3.4-1968 iso-ir-6 ANSI_X3.4-1986 ISO_646.irv:1991 ASCII
Expand Down Expand Up @@ -12472,7 +12472,7 @@ if {[tk windowingsystem] eq "aqua"} {

catch {
# follow the XDG base directory specification by default. See
# http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
# https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html
if {[info exists env(XDG_CONFIG_HOME)] && $env(XDG_CONFIG_HOME) ne ""} {
# XDG_CONFIG_HOME environment variable is set
set config_file [file join $env(XDG_CONFIG_HOME) git gitk]
Expand Down
2 changes: 1 addition & 1 deletion gitweb/INSTALL
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ You can specify the following configuration variables when building GIT:
created. [Default: /etc/gitweb.conf]
* HIGHLIGHT_BIN
Path to the highlight executable to use (must be the one from
http://www.andre-simon.de due to assumptions about parameters and output).
http://andre-simon.de/zip/download.php due to assumptions about parameters and output).
Useful if highlight is not installed on your webserver's PATH.
[Default: highlight]

Expand Down
10 changes: 5 additions & 5 deletions gitweb/gitweb.perl
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,9 @@ sub evaluate_uri {
our $javascript = "++GITWEB_JS++";

# URI and label (title) of GIT logo link
#our $logo_url = "http://www.kernel.org/pub/software/scm/git/docs/";
#our $logo_url = "https://www.kernel.org/pub/software/scm/git/docs/";
#our $logo_label = "git documentation";
our $logo_url = "http://git-scm.com/";
our $logo_url = "https://git-scm.com/";
our $logo_label = "git homepage";

# source of projects list
Expand Down Expand Up @@ -197,7 +197,7 @@ sub evaluate_uri {
our $prevent_xss = 0;

# Path to the highlight executable to use (must be the one from
# http://www.andre-simon.de due to assumptions about parameters and output).
# http://andre-simon.de/zip/download.php due to assumptions about parameters and output).
# Useful if highlight is not installed on your webserver's PATH.
# [Default: highlight]
our $highlight_bin = "++HIGHLIGHT_BIN++";
Expand Down Expand Up @@ -269,7 +269,7 @@ sub evaluate_uri {
# Leave it undefined (or set to 'undef') to turn off load checking.
our $maxload = 300;

# configuration for 'highlight' (http://www.andre-simon.de/)
# configuration for 'highlight' (http://andre-simon.de/doku/highlight/en/highlight.php)
# match by basename
our %highlight_basename = (
#'Program' => 'py',
Expand Down Expand Up @@ -8193,7 +8193,7 @@ sub git_feed {
my $have_blame = gitweb_check_feature('blame');

# Atom: http://www.atomenabled.org/developers/syndication/
# RSS: http://www.notestips.com/80256B3A007F2692/1/NAMO5P9UPQ
# RSS: https://web.archive.org/web/20030729001534/http://www.notestips.com/80256B3A007F2692/1/NAMO5P9UPQ
if ($format ne 'rss' && $format ne 'atom') {
die_error(400, "Unknown web feed format");
}
Expand Down
2 changes: 1 addition & 1 deletion gitweb/static/gitweb.css
Original file line number Diff line number Diff line change
Expand Up @@ -667,7 +667,7 @@ div.remote {
}


/* Style definition generated by highlight 2.4.5, http://www.andre-simon.de/ */
/* Style definition generated by highlight 2.4.5, http://andre-simon.de/doku/highlight/en/highlight.php */

/* Highlighting theme definition: */

Expand Down
10 changes: 5 additions & 5 deletions gitweb/static/js/lib/common-lib.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,8 +123,8 @@ function addCssRule(selector, style) {
* NOTE that there are limits and differences compared to native
* getElementsByClassName as defined by e.g.:
* https://developer.mozilla.org/en/DOM/document.getElementsByClassName
* http://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#dom-getelementsbyclassname
* http://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#dom-document-getelementsbyclassname
* https://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#dom-getelementsbyclassname
* https://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#dom-document-getelementsbyclassname
*
* Namely, this implementation supports only single class name as
* argument and not set of space-separated tokens representing classes,
Expand All @@ -133,11 +133,11 @@ function addCssRule(selector, style) {
* (via getElementsByTagName).
*
* Based on
* http://code.google.com/p/getelementsbyclassname/
* https://code.google.com/p/getelementsbyclassname/
* http://www.dustindiaz.com/getelementsbyclass/
* http://stackoverflow.com/questions/1818865/do-we-have-getelementsbyclassname-in-javascript
* https://stackoverflow.com/questions/1818865/do-we-have-getelementsbyclassname-in-javascript
*
* See also http://ejohn.org/blog/getelementsbyclassname-speed-comparison/
* See also https://johnresig.com/blog/getelementsbyclassname-speed-comparison/
*
* @param {String} class: name of _single_ class to find
* @param {String} [taghint] limit search to given tags
Expand Down
Loading