Re: [PATCH v2 08/11] docs: refcount-vs-atomic.rst: prefix url with https

From: Mauro Carvalho Chehab
Date: Wed May 09 2018 - 10:41:09 EST


Em Wed, 9 May 2018 15:43:19 +0200
Andrea Parri <andrea.parri@xxxxxxxxxxxxxxxxxxxx> escreveu:

> Hi Mauro,
>
> On Wed, May 09, 2018 at 10:18:51AM -0300, Mauro Carvalho Chehab wrote:
> > There's a github URL there, but it is not prefixed by https.
> > Add a prefix, to avoid false-positives with:
> > ./scripts/documentation-file-ref-check
> >
> > As a side effect, Sphinx should also generate a cross-ref.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx>
>
> There seems to be a "conflicting" patch ("applied" according to Jon):
>
> http://lkml.kernel.org/r/1525468309-5310-1-git-send-email-andrea.parri@xxxxxxxxxxxxxxxxxxxx
>
> Let me stress here that the github repo. is out-of-date (and we have
> no plans to keep that in sync with mainline).

Good! One less patch pending :-)

Yeah, it seems that Jon pushed it from his repository today. I did a new
git remote update and I'm now seeing this one too.

>
> Andrea
>
>
> > ---
> > Documentation/core-api/refcount-vs-atomic.rst | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/core-api/refcount-vs-atomic.rst b/Documentation/core-api/refcount-vs-atomic.rst
> > index 83351c258cdb..185d659e350a 100644
> > --- a/Documentation/core-api/refcount-vs-atomic.rst
> > +++ b/Documentation/core-api/refcount-vs-atomic.rst
> > @@ -17,7 +17,7 @@ in order to help maintainers validate their code against the change in
> > these memory ordering guarantees.
> >
> > The terms used through this document try to follow the formal LKMM defined in
> > -github.com/aparri/memory-model/blob/master/Documentation/explanation.txt
> > +https://github.com/aparri/memory-model/blob/master/Documentation/explanation.txt
> >
> > memory-barriers.txt and atomic_t.txt provide more background to the
> > memory ordering in general and for atomic operations specifically.
> > --
> > 2.17.0
> >



Thanks,
Mauro