[PATCH 0/2] Enable hyperlink border in bibliography

[PATCH 0/2] Enable hyperlink border in bibliography

[Akira Yokosawa]

From 5a3db5ddd9458a1787bdf9bd554146079501df18 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 3 Mar 2017 21:21:25 +0900
Subject: [PATCH 0/2] Enable hyperlink border in bibliography

So, this patch set enables hyperlink border for all layouts.
Patch 2 is another update to append doi.

                                Thanks, Akira
--
Akira Yokosawa (2):
  appendix: Enable hyperlink border in bibliography for all layouts
  bib/syncrefs: Add doi to Hoare74

 appendix/appendix.tex | 4 ++--
 bib/syncrefs.bib      | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

-- 
2.7.4

[PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts

[Akira Yokosawa]

From ec794a1bd368c5ad381fac0594a254fd6a23f619 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 3 Mar 2017 20:56:59 +0900
Subject: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts

"pdfborder" in bibliography also helps in identifying known broken
urls. This commit enables the border regardless of layout.

Suggested-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 appendix/appendix.tex | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/appendix/appendix.tex b/appendix/appendix.tex
index e17c5e1..63bf728 100644
--- a/appendix/appendix.tex
+++ b/appendix/appendix.tex
@@ -11,12 +11,12 @@
   \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
 }{
   \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
-  \hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
 }
+\hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
 \bibliography{bib/RCU,bib/WFS,bib/hw,bib/os,bib/parallelsys,bib/patterns,bib/perfmeas,bib/refs,bib/syncrefs,bib/search,bib/swtools,bib/realtime,bib/TM,bib/standards,bib/OSS,bib/maze,bib/energy}
+\hypersetup{pdfborder=0 0 0}
 \IfTwoColumn{
   \end{adjustwidth*}\twocolumn
 }{
-  \hypersetup{pdfborder=0 0 0}
 }
 \include{appendix/ack/ack}
-- 
2.7.4

Re: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts

[Akira Yokosawa]

On 2017/03/03 21:28, Akira Yokosawa wrote:
>>From ec794a1bd368c5ad381fac0594a254fd6a23f619 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Fri, 3 Mar 2017 20:56:59 +0900
> Subject: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts
> 
> "pdfborder" in bibliography also helps in identifying known broken
> urls. This commit enables the border regardless of layout.

One thing to note:

If a hyperlink crosses page boundary, the hyperlink is wrongly embedded
also in the footer and header strings. As of now, target "1csf" hits this
issue at the bottom of first page of Bibliography (p. 653-654).
This is a well known issue of "hyperref" package, which would cause
a build error in 2c layout. Before enabling the border, we just didn't
notice the issue.
Other target might hit this issue by changes made in bibliography.
Are you OK with this possible ugliness?

                               Thanks, Akira

> 
> Suggested-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> ---
>  appendix/appendix.tex | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/appendix/appendix.tex b/appendix/appendix.tex
> index e17c5e1..63bf728 100644
> --- a/appendix/appendix.tex
> +++ b/appendix/appendix.tex
> @@ -11,12 +11,12 @@
>    \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
>  }{
>    \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
> -  \hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
>  }
> +\hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
>  \bibliography{bib/RCU,bib/WFS,bib/hw,bib/os,bib/parallelsys,bib/patterns,bib/perfmeas,bib/refs,bib/syncrefs,bib/search,bib/swtools,bib/realtime,bib/TM,bib/standards,bib/OSS,bib/maze,bib/energy}
> +\hypersetup{pdfborder=0 0 0}
>  \IfTwoColumn{
>    \end{adjustwidth*}\twocolumn
>  }{
> -  \hypersetup{pdfborder=0 0 0}
>  }
>  \include{appendix/ack/ack}
> 

Re: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts

[Paul E. McKenney]

On Sat, Mar 04, 2017 at 12:25:59AM +0900, Akira Yokosawa wrote:
> On 2017/03/03 21:28, Akira Yokosawa wrote:
> >>From ec794a1bd368c5ad381fac0594a254fd6a23f619 Mon Sep 17 00:00:00 2001
> > From: Akira Yokosawa <akiyks@gmail.com>
> > Date: Fri, 3 Mar 2017 20:56:59 +0900
> > Subject: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts
> > 
> > "pdfborder" in bibliography also helps in identifying known broken
> > urls. This commit enables the border regardless of layout.
> 
> One thing to note:
> 
> If a hyperlink crosses page boundary, the hyperlink is wrongly embedded
> also in the footer and header strings. As of now, target "1csf" hits this
> issue at the bottom of first page of Bibliography (p. 653-654).
> This is a well known issue of "hyperref" package, which would cause
> a build error in 2c layout. Before enabling the border, we just didn't
> notice the issue.
> Other target might hit this issue by changes made in bibliography.
> Are you OK with this possible ugliness?

Seems like a good diagnostic.  It might be necessary to manually adjust
the .bbl file for editions, and perhaps also for releases.  One nice
thing is that this change gives visual indication as to where adjustment
is needed.

Queued, will be pushed when I next get decent connectivity, thank you!

							Thanx, Paul

>                                Thanks, Akira
> 
> > 
> > Suggested-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> > Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> > ---
> >  appendix/appendix.tex | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/appendix/appendix.tex b/appendix/appendix.tex
> > index e17c5e1..63bf728 100644
> > --- a/appendix/appendix.tex
> > +++ b/appendix/appendix.tex
> > @@ -11,12 +11,12 @@
> >    \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
> >  }{
> >    \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option
> > -  \hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
> >  }
> > +\hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1}
> >  \bibliography{bib/RCU,bib/WFS,bib/hw,bib/os,bib/parallelsys,bib/patterns,bib/perfmeas,bib/refs,bib/syncrefs,bib/search,bib/swtools,bib/realtime,bib/TM,bib/standards,bib/OSS,bib/maze,bib/energy}
> > +\hypersetup{pdfborder=0 0 0}
> >  \IfTwoColumn{
> >    \end{adjustwidth*}\twocolumn
> >  }{
> > -  \hypersetup{pdfborder=0 0 0}
> >  }
> >  \include{appendix/ack/ack}
> > 
> 

[PATCH 2/2] bib/syncrefs: Add doi to Hoare74

[Akira Yokosawa]

From 5a3db5ddd9458a1787bdf9bd554146079501df18 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 3 Mar 2017 21:20:05 +0900
Subject: [PATCH 2/2] bib/syncrefs: Add doi to Hoare74

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 bib/syncrefs.bib | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bib/syncrefs.bib b/bib/syncrefs.bib
index 0a946d4..c5d7372 100644
--- a/bib/syncrefs.bib
+++ b/bib/syncrefs.bib
@@ -268,6 +268,7 @@ Brent A Kingsbury"
 ,volume="17"
 ,number="10"
 ,pages="549-557"
+,doi="10.1145/355620.361161"
 }

 @article{Lamport74a
-- 
2.7.4

Re: [PATCH 2/2] bib/syncrefs: Add doi to Hoare74

[Paul E. McKenney]

On Fri, Mar 03, 2017 at 09:29:46PM +0900, Akira Yokosawa wrote:
> >From 5a3db5ddd9458a1787bdf9bd554146079501df18 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Fri, 3 Mar 2017 21:20:05 +0900
> Subject: [PATCH 2/2] bib/syncrefs: Add doi to Hoare74
> 
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>

Applied, thank you!  I will push this as soon as I get to decent
internet connectivity.

							Thanx, Paul

> ---
>  bib/syncrefs.bib | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/bib/syncrefs.bib b/bib/syncrefs.bib
> index 0a946d4..c5d7372 100644
> --- a/bib/syncrefs.bib
> +++ b/bib/syncrefs.bib
> @@ -268,6 +268,7 @@ Brent A Kingsbury"
>  ,volume="17"
>  ,number="10"
>  ,pages="549-557"
> +,doi="10.1145/355620.361161"
>  }
> 
>  @article{Lamport74a
> -- 
> 2.7.4
> 
>