All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] Enable hyperlink border in bibliography
@ 2017-03-03 12:27 Akira Yokosawa
  2017-03-03 12:28 ` [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts Akira Yokosawa
  2017-03-03 12:29 ` [PATCH 2/2] bib/syncrefs: Add doi to Hoare74 Akira Yokosawa
  0 siblings, 2 replies; 6+ messages in thread
From: Akira Yokosawa @ 2017-03-03 12:27 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, 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


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts
  2017-03-03 12:27 [PATCH 0/2] Enable hyperlink border in bibliography Akira Yokosawa
@ 2017-03-03 12:28 ` Akira Yokosawa
  2017-03-03 15:25   ` Akira Yokosawa
  2017-03-03 12:29 ` [PATCH 2/2] bib/syncrefs: Add doi to Hoare74 Akira Yokosawa
  1 sibling, 1 reply; 6+ messages in thread
From: Akira Yokosawa @ 2017-03-03 12:28 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, 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



^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH 2/2] bib/syncrefs: Add doi to Hoare74
  2017-03-03 12:27 [PATCH 0/2] Enable hyperlink border in bibliography Akira Yokosawa
  2017-03-03 12:28 ` [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts Akira Yokosawa
@ 2017-03-03 12:29 ` Akira Yokosawa
  2017-03-05 21:24   ` Paul E. McKenney
  1 sibling, 1 reply; 6+ messages in thread
From: Akira Yokosawa @ 2017-03-03 12:29 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, 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



^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts
  2017-03-03 12:28 ` [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts Akira Yokosawa
@ 2017-03-03 15:25   ` Akira Yokosawa
  2017-03-05 21:20     ` Paul E. McKenney
  0 siblings, 1 reply; 6+ messages in thread
From: Akira Yokosawa @ 2017-03-03 15:25 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, 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}
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts
  2017-03-03 15:25   ` Akira Yokosawa
@ 2017-03-05 21:20     ` Paul E. McKenney
  0 siblings, 0 replies; 6+ messages in thread
From: Paul E. McKenney @ 2017-03-05 21:20 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

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}
> > 
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] bib/syncrefs: Add doi to Hoare74
  2017-03-03 12:29 ` [PATCH 2/2] bib/syncrefs: Add doi to Hoare74 Akira Yokosawa
@ 2017-03-05 21:24   ` Paul E. McKenney
  0 siblings, 0 replies; 6+ messages in thread
From: Paul E. McKenney @ 2017-03-05 21:24 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

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
> 
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-06 10:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-03 12:27 [PATCH 0/2] Enable hyperlink border in bibliography Akira Yokosawa
2017-03-03 12:28 ` [PATCH 1/2] appendix: Enable hyperlink border in bibliography for all layouts Akira Yokosawa
2017-03-03 15:25   ` Akira Yokosawa
2017-03-05 21:20     ` Paul E. McKenney
2017-03-03 12:29 ` [PATCH 2/2] bib/syncrefs: Add doi to Hoare74 Akira Yokosawa
2017-03-05 21:24   ` Paul E. McKenney

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.