From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Akira Yokosawa <akiyks@gmail.com>
Cc: perfbook@vger.kernel.org
Subject: Re: [PATCH 0/2] Enable inlinelinks of alphapf.bst for 1c
Date: Wed, 28 Dec 2016 21:21:03 -0800 [thread overview]
Message-ID: <20161229052103.GE3742@linux.vnet.ibm.com> (raw)
In-Reply-To: <adbe2696-8bcf-1fbf-7395-3b84351e77ba@gmail.com>
On Thu, Dec 29, 2016 at 02:10:39PM +0900, Akira Yokosawa wrote:
> On 2016/12/28 21:06:01 -0800, Paul E. McKenney wrote:
> > On Thu, Dec 29, 2016 at 01:13:24PM +0900, Akira Yokosawa wrote:
> >> On 2016/12/28 19:56:03 -0800, Paul E. McKenney wrote:
> >>> On Wed, Dec 28, 2016 at 07:07:28AM +0900, Akira Yokosawa wrote:
> >>>> On 2016/12/27 12:53:45 -0800, Paul E. McKenney wrote:
> >>>>> On Wed, Dec 28, 2016 at 12:25:48AM +0900, Akira Yokosawa wrote:
> >>>>>> >From 3f4f73d738b77422bd074322f3ac67f644a8d580 Mon Sep 17 00:00:00 2001
> >>>>>> From: Akira Yokosawa <akiyks@gmail.com>
> >>>>>> Date: Wed, 28 Dec 2016 00:10:28 +0900
> >>>>>> Subject: [PATCH 0/2] Enable inlinelinks of alphapf.bst for 1c
> >>>>>>
> >>>>>> Hi Paul,
> >>>>>>
> >>>>>> On 2016/12/26 16:39:52 -0800, Paul E. McKenney wrote:
> >>>>>>> On Mon, Dec 26, 2016 at 11:52:40PM +0900, Akira Yokosawa wrote:
> >>>>>> [snip]
> >>>>>>>> One idea is to use alphapf.bst only for 1c with inlinelinks enabled.
> >>>>>>>>
> >>>>>>>> Thoughts?
> >>>>>>>
> >>>>>>> I am good with enabling alphaph.bst inlinelinks only for 1c!
> >>>>>>
> >>>>>> So, this patch set does the change.
> >>>>>> For 2c, we reverted to the original alpha bibliography style for the
> >>>>>> moment.
> >>>>>> And I'll keep polishing the band-aid script to shorten the href-ed part
> >>>>>> in titles. You might want to try it for 2c when you do a release.
> >>>>>
> >>>>> Applied and pushed, thank you!
> >>>>>
> >>>>> Speaking of releases, for whatever it is worth, I expect to do one
> >>>>> in a few days.
> >>>>
> >>>> Every 6 month!
> >>>>
> >>>> At the current state of ongoing bibliography update, I don't think
> >>>> it makes much difference if you try inlinelinks for 2c.
> >>>> Or, will you apply the "bib-append-dio" series this time?
> >>>> Then it might be worthwhile. You can replace the released .pdf with
> >>>> inlinelinks enabled after the fact.
> >>>>
> >>>> I have readied other branches of url updates of .bib files, but I thought
> >>>> I should wait and send their pull request after the doi updates are applied.
> >>>> If you are willing to apply the other (lot of) updates for the next release,
> >>>> I can send the pull requests soon. But it is not urgent.
> >>>>
> >>>> By the way, I want the "courier scaled" font to be default monospace
> >>>> font for 2c and 1c builds. Have you tried the "mss" target lately?
> >>>> If you are ok with it, I'll prepare a patch to do so.
> >>>
> >>> It does build, but what am I looking for? Does it allow me to remove
> >>> the bogus "~" characters in some of the tables containing \co{}?
> >>
> >> Well, that may be possible, but for now, I want you to see if the
> >> reduced height of the monospace font in the text is acceptable for you.
> >
> > It looks fine to me.
> >
> >> The direct effect of the change is that we can revert 2 commits
> >> which manually specified smaller font sizes for code sample figures.
> >>
> >> See commits bc0fd9d40035 ("after: Tweak font size of Figure A.2") and
> >> c7c5e8a1829e ("formal/dyntickrcu: Adjust font size of sample code").
> >>
> >> Undoing tweaks in Tables can be looked at once we switch the default font.
> >
> > However, this sounds like something to do immediately after release in
> > order to give people more time to spot any problems. Seem reasonable?
>
> Yes, I agree.
> Will submit a patch set after the upcoming release.
Very good!
Thanx, Paul
> >>>>>> --
> >>>>>> Akira Yokosawa (2):
> >>>>>> Makefile: Fix .bbl dependency
> >>>>>> Enable 'inlinelinks' option of alphapf.bst for 1c layout
> >>>>>>
> >>>>>> Makefile | 4 ++--
> >>>>>> alphapf.bst | 2 +-
> >>>>>> appendix/appendix.tex | 6 +++++-
> >>>>>> 3 files changed, 8 insertions(+), 4 deletions(-)
> >>>>>>
> >>>>>> --
> >>>>>> 2.7.4
> >>>>>>
> >>>>>
> >>>>>
> >>>>
> >>>
> >>>
> >> --
> >> To unsubscribe from this list: send the line "unsubscribe perfbook" in
> >> the body of a message to majordomo@vger.kernel.org
> >> More majordomo info at http://vger.kernel.org/majordomo-info.html
> >>
> >
> >
>
next prev parent reply other threads:[~2016-12-29 5:21 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-27 15:25 [PATCH 0/2] Enable inlinelinks of alphapf.bst for 1c Akira Yokosawa
2016-12-27 15:27 ` [PATCH 1/2] Makefile: Fix .bbl dependency Akira Yokosawa
2016-12-27 15:30 ` [PATCH 2/2] Enable 'inlinelinks' option of alphapf.bst for 1c layout Akira Yokosawa
2016-12-27 20:53 ` [PATCH 0/2] Enable inlinelinks of alphapf.bst for 1c Paul E. McKenney
2016-12-27 22:07 ` Akira Yokosawa
2016-12-29 3:56 ` Paul E. McKenney
2016-12-29 4:13 ` Akira Yokosawa
2016-12-29 5:06 ` Paul E. McKenney
2016-12-29 5:10 ` Akira Yokosawa
2016-12-29 5:21 ` Paul E. McKenney [this message]
2016-12-30 15:33 ` Akira Yokosawa
2016-12-30 16:59 ` Paul E. McKenney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161229052103.GE3742@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akiyks@gmail.com \
--cc=perfbook@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox