All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: Fix typo "baloon" -> "balloon"
@ 2016-05-19 15:01 Akira Yokosawa
  2016-05-19 15:06 ` [PATCH] Improve messages regarding font installation Akira Yokosawa
  0 siblings, 1 reply; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-19 15:01 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

Hi, Paul.

Thank you for fixing this.
However, it breaks adjustment of "##" characters in the warning messages.
I made some improvement regarding the font installation checks.

I'm going to send the patch in reply to this message.

                                         Thanks, Akira


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH] Improve messages regarding font installation
  2016-05-19 15:01 Fix typo "baloon" -> "balloon" Akira Yokosawa
@ 2016-05-19 15:06 ` Akira Yokosawa
  2016-05-19 19:46   ` Paul E. McKenney
  0 siblings, 1 reply; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-19 15:06 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Thu, 19 May 2016 23:34:27 +0900
Subject: [PATCH] Improve messages regarding font installation

In commit d8318d8db866 ("Add font installation check"), there was
typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
"balloon"").
However, it breaks adjustment of "##" characters in the warning
messages.
This commit fixes them.
It also improves explanation of item 1 in FAQ-BUILD.txt by adding
a reference to item 1 of FAQ.txt.
It also adds a comment in Figure 1.1 that refers to item 1 in
FAQ.txt

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 FAQ-BUILD.txt         | 6 +++---
 howto/howto.tex       | 9 +++++----
 utilities/eps2pdf.sh  | 2 +-
 utilities/runlatex.sh | 2 +-
 4 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
index d963534..e888725 100644
--- a/FAQ-BUILD.txt
+++ b/FAQ-BUILD.txt
@@ -1,6 +1,6 @@
-1.	I have installed the Steel City Comic font, and rebuilt
-	"perfbook.pdf".  But it doesn't seem to have any effect.
-	What am I missing?
+1.	I have installed the Steel City Comic font referring to item 1
+	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
+	have any effect. What am I missing?

 	A.	If you built "perfbook.pdf" before installing the font,
 		you need to rebuild cartoons whose sources are .svg
diff --git a/howto/howto.tex b/howto/howto.tex
index 6980114..7a70bb4 100644
--- a/howto/howto.tex
+++ b/howto/howto.tex
@@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
 \begin{verbatim}
   1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
   2 cd perfbook
-  3 make
-  4 evince perfbook.pdf & # Two-column version
-  5 make perfbook-1c.pdf
-  6 evince perfbook-1c.pdf & # One-column version for e-readers
+  3 # You may need to install a font here. See item 1 in FAQ.txt.
+  4 make
+  5 evince perfbook.pdf & # Two-column version
+  6 make perfbook-1c.pdf
+  7 evince perfbook-1c.pdf & # One-column version for e-readers
 \end{verbatim}
 }
 \caption{Creating an Up-To-Date PDF}
diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
index 1497bdd..0a82d57 100644
--- a/utilities/eps2pdf.sh
+++ b/utilities/eps2pdf.sh
@@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
 then
 	echo "#######################################################################"
 	echo "## Steel City Comic font is not found in the font cache!             ##"
-	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
+	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
 	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
 	echo "## Nevertheless, this build will resume in a short while.            ##"
 	echo "#######################################################################"
diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
index 9ae9021..94d46d6 100644
--- a/utilities/runlatex.sh
+++ b/utilities/runlatex.sh
@@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
 then
 	echo "#######################################################################"
 	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
-	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
+	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
 	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
 	echo "#######################################################################"
 fi
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-19 15:06 ` [PATCH] Improve messages regarding font installation Akira Yokosawa
@ 2016-05-19 19:46   ` Paul E. McKenney
  2016-05-19 22:18     ` Akira Yokosawa
  0 siblings, 1 reply; 10+ messages in thread
From: Paul E. McKenney @ 2016-05-19 19:46 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Thu, 19 May 2016 23:34:27 +0900
> Subject: [PATCH] Improve messages regarding font installation
> 
> In commit d8318d8db866 ("Add font installation check"), there was
> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
> "balloon"").
> However, it breaks adjustment of "##" characters in the warning
> messages.
> This commit fixes them.
> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
> a reference to item 1 of FAQ.txt.
> It also adds a comment in Figure 1.1 that refers to item 1 in
> FAQ.txt
> 
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>

Queued, thank you!

BTW, I am still getting this:

#######################################################################
## Steel City Comic font is not found in the resulting PDF!          ##
## Some speech balloons in the cartoons have been rendered awkwardly.##
## See item 1 in FAQ-BUILD.txt for the solution.                     ##
#######################################################################

I do have some fonts installed:

$ fc-list | grep steel
/usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta

This is from steel-city-comic.regular.ttf.  Is there something more that
I need?  (I do plead much ignorance on fonts.)

							Thanx, Paul

> ---
>  FAQ-BUILD.txt         | 6 +++---
>  howto/howto.tex       | 9 +++++----
>  utilities/eps2pdf.sh  | 2 +-
>  utilities/runlatex.sh | 2 +-
>  4 files changed, 10 insertions(+), 9 deletions(-)
> 
> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
> index d963534..e888725 100644
> --- a/FAQ-BUILD.txt
> +++ b/FAQ-BUILD.txt
> @@ -1,6 +1,6 @@
> -1.	I have installed the Steel City Comic font, and rebuilt
> -	"perfbook.pdf".  But it doesn't seem to have any effect.
> -	What am I missing?
> +1.	I have installed the Steel City Comic font referring to item 1
> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
> +	have any effect. What am I missing?
> 
>  	A.	If you built "perfbook.pdf" before installing the font,
>  		you need to rebuild cartoons whose sources are .svg
> diff --git a/howto/howto.tex b/howto/howto.tex
> index 6980114..7a70bb4 100644
> --- a/howto/howto.tex
> +++ b/howto/howto.tex
> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
>  \begin{verbatim}
>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
>    2 cd perfbook
> -  3 make
> -  4 evince perfbook.pdf & # Two-column version
> -  5 make perfbook-1c.pdf
> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
> +  4 make
> +  5 evince perfbook.pdf & # Two-column version
> +  6 make perfbook-1c.pdf
> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
>  \end{verbatim}
>  }
>  \caption{Creating an Up-To-Date PDF}
> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
> index 1497bdd..0a82d57 100644
> --- a/utilities/eps2pdf.sh
> +++ b/utilities/eps2pdf.sh
> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
>  then
>  	echo "#######################################################################"
>  	echo "## Steel City Comic font is not found in the font cache!             ##"
> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
>  	echo "## Nevertheless, this build will resume in a short while.            ##"
>  	echo "#######################################################################"
> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
> index 9ae9021..94d46d6 100644
> --- a/utilities/runlatex.sh
> +++ b/utilities/runlatex.sh
> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
>  then
>  	echo "#######################################################################"
>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
>  	echo "#######################################################################"
>  fi
> -- 
> 1.9.1
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-19 19:46   ` Paul E. McKenney
@ 2016-05-19 22:18     ` Akira Yokosawa
  2016-05-19 22:42       ` Akira Yokosawa
  0 siblings, 1 reply; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-19 22:18 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

On 2016/05/19 12:46:51 -0700, Paul E. McKenney wrote:
> On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
>> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
>> From: Akira Yokosawa <akiyks@gmail.com>
>> Date: Thu, 19 May 2016 23:34:27 +0900
>> Subject: [PATCH] Improve messages regarding font installation
>>
>> In commit d8318d8db866 ("Add font installation check"), there was
>> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
>> "balloon"").
>> However, it breaks adjustment of "##" characters in the warning
>> messages.
>> This commit fixes them.
>> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
>> a reference to item 1 of FAQ.txt.
>> It also adds a comment in Figure 1.1 that refers to item 1 in
>> FAQ.txt
>>
>> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> 
> Queued, thank you!
> 
> BTW, I am still getting this:
> 
> #######################################################################
> ## Steel City Comic font is not found in the resulting PDF!          ##
> ## Some speech balloons in the cartoons have been rendered awkwardly.##
> ## See item 1 in FAQ-BUILD.txt for the solution.                     ##
> #######################################################################
> 
> I do have some fonts installed:
> 
> $ fc-list | grep steel
> /usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta
> 
> This is from steel-city-comic.regular.ttf.  Is there something more that
> I need?  (I do plead much ignorance on fonts.)

I didn't expect you to hit this...

You mean the characters in Figures 3.1 and 3.2 don't fit in the balloons?

"make touchsvg; make" should help you, but I think you've already tried it.

"make neatfreak; make" will rebuild all the pdf files, but I don't expect
any difference in the result.

Could you send me "cartoons/r-2014-Old-man-and-Brat.pdf" off the list?

                                                    Thanks, Akira

> 
> 							Thanx, Paul
> 
>> ---
>>  FAQ-BUILD.txt         | 6 +++---
>>  howto/howto.tex       | 9 +++++----
>>  utilities/eps2pdf.sh  | 2 +-
>>  utilities/runlatex.sh | 2 +-
>>  4 files changed, 10 insertions(+), 9 deletions(-)
>>
>> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
>> index d963534..e888725 100644
>> --- a/FAQ-BUILD.txt
>> +++ b/FAQ-BUILD.txt
>> @@ -1,6 +1,6 @@
>> -1.	I have installed the Steel City Comic font, and rebuilt
>> -	"perfbook.pdf".  But it doesn't seem to have any effect.
>> -	What am I missing?
>> +1.	I have installed the Steel City Comic font referring to item 1
>> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
>> +	have any effect. What am I missing?
>>
>>  	A.	If you built "perfbook.pdf" before installing the font,
>>  		you need to rebuild cartoons whose sources are .svg
>> diff --git a/howto/howto.tex b/howto/howto.tex
>> index 6980114..7a70bb4 100644
>> --- a/howto/howto.tex
>> +++ b/howto/howto.tex
>> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
>>  \begin{verbatim}
>>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
>>    2 cd perfbook
>> -  3 make
>> -  4 evince perfbook.pdf & # Two-column version
>> -  5 make perfbook-1c.pdf
>> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
>> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
>> +  4 make
>> +  5 evince perfbook.pdf & # Two-column version
>> +  6 make perfbook-1c.pdf
>> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
>>  \end{verbatim}
>>  }
>>  \caption{Creating an Up-To-Date PDF}
>> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
>> index 1497bdd..0a82d57 100644
>> --- a/utilities/eps2pdf.sh
>> +++ b/utilities/eps2pdf.sh
>> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
>>  then
>>  	echo "#######################################################################"
>>  	echo "## Steel City Comic font is not found in the font cache!             ##"
>> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
>> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
>>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
>>  	echo "## Nevertheless, this build will resume in a short while.            ##"
>>  	echo "#######################################################################"
>> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
>> index 9ae9021..94d46d6 100644
>> --- a/utilities/runlatex.sh
>> +++ b/utilities/runlatex.sh
>> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
>>  then
>>  	echo "#######################################################################"
>>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
>> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
>> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
>>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
>>  	echo "#######################################################################"
>>  fi
>> -- 
>> 1.9.1
>>
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-19 22:18     ` Akira Yokosawa
@ 2016-05-19 22:42       ` Akira Yokosawa
  2016-05-20  4:13         ` Paul E. McKenney
  0 siblings, 1 reply; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-19 22:42 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

On 2016/05/20 07:18 +0900, Akira Yokosawa wrote:
> On 2016/05/19 12:46:51 -0700, Paul E. McKenney wrote:
>> On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
>>> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
>>> From: Akira Yokosawa <akiyks@gmail.com>
>>> Date: Thu, 19 May 2016 23:34:27 +0900
>>> Subject: [PATCH] Improve messages regarding font installation
>>>
>>> In commit d8318d8db866 ("Add font installation check"), there was
>>> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
>>> "balloon"").
>>> However, it breaks adjustment of "##" characters in the warning
>>> messages.
>>> This commit fixes them.
>>> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
>>> a reference to item 1 of FAQ.txt.
>>> It also adds a comment in Figure 1.1 that refers to item 1 in
>>> FAQ.txt
>>>
>>> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
>>
>> Queued, thank you!
>>
>> BTW, I am still getting this:
>>
>> #######################################################################
>> ## Steel City Comic font is not found in the resulting PDF!          ##
>> ## Some speech balloons in the cartoons have been rendered awkwardly.##
>> ## See item 1 in FAQ-BUILD.txt for the solution.                     ##
>> #######################################################################
>>
>> I do have some fonts installed:
>>
>> $ fc-list | grep steel
>> /usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta
>>
>> This is from steel-city-comic.regular.ttf.  Is there something more that
>> I need?  (I do plead much ignorance on fonts.)
> 
> I didn't expect you to hit this...
> 
> You mean the characters in Figures 3.1 and 3.2 don't fit in the balloons?
> 
> "make touchsvg; make" should help you, but I think you've already tried it.
> 
> "make neatfreak; make" will rebuild all the pdf files, but I don't expect
> any difference in the result.
> 
> Could you send me "cartoons/r-2014-Old-man-and-Brat.pdf" off the list?
> 
>                                                     Thanks, Akira

Or do you happen to have some environment variable set to make grep ignore
binary files, don't you?

Then the grep in runlatex.sh needs to override it. I'll see what can be done.

> 
>>
>> 							Thanx, Paul
>>
>>> ---
>>>  FAQ-BUILD.txt         | 6 +++---
>>>  howto/howto.tex       | 9 +++++----
>>>  utilities/eps2pdf.sh  | 2 +-
>>>  utilities/runlatex.sh | 2 +-
>>>  4 files changed, 10 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
>>> index d963534..e888725 100644
>>> --- a/FAQ-BUILD.txt
>>> +++ b/FAQ-BUILD.txt
>>> @@ -1,6 +1,6 @@
>>> -1.	I have installed the Steel City Comic font, and rebuilt
>>> -	"perfbook.pdf".  But it doesn't seem to have any effect.
>>> -	What am I missing?
>>> +1.	I have installed the Steel City Comic font referring to item 1
>>> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
>>> +	have any effect. What am I missing?
>>>
>>>  	A.	If you built "perfbook.pdf" before installing the font,
>>>  		you need to rebuild cartoons whose sources are .svg
>>> diff --git a/howto/howto.tex b/howto/howto.tex
>>> index 6980114..7a70bb4 100644
>>> --- a/howto/howto.tex
>>> +++ b/howto/howto.tex
>>> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
>>>  \begin{verbatim}
>>>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
>>>    2 cd perfbook
>>> -  3 make
>>> -  4 evince perfbook.pdf & # Two-column version
>>> -  5 make perfbook-1c.pdf
>>> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
>>> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
>>> +  4 make
>>> +  5 evince perfbook.pdf & # Two-column version
>>> +  6 make perfbook-1c.pdf
>>> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
>>>  \end{verbatim}
>>>  }
>>>  \caption{Creating an Up-To-Date PDF}
>>> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
>>> index 1497bdd..0a82d57 100644
>>> --- a/utilities/eps2pdf.sh
>>> +++ b/utilities/eps2pdf.sh
>>> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
>>>  then
>>>  	echo "#######################################################################"
>>>  	echo "## Steel City Comic font is not found in the font cache!             ##"
>>> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
>>> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
>>>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
>>>  	echo "## Nevertheless, this build will resume in a short while.            ##"
>>>  	echo "#######################################################################"
>>> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
>>> index 9ae9021..94d46d6 100644
>>> --- a/utilities/runlatex.sh
>>> +++ b/utilities/runlatex.sh
>>> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
>>>  then
>>>  	echo "#######################################################################"
>>>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
>>> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
>>> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
>>>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
>>>  	echo "#######################################################################"
>>>  fi
>>> -- 
>>> 1.9.1
>>>
>>
>>
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-19 22:42       ` Akira Yokosawa
@ 2016-05-20  4:13         ` Paul E. McKenney
  2016-05-20 12:00           ` Akira Yokosawa
  0 siblings, 1 reply; 10+ messages in thread
From: Paul E. McKenney @ 2016-05-20  4:13 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

On Fri, May 20, 2016 at 07:42:53AM +0900, Akira Yokosawa wrote:
> On 2016/05/20 07:18 +0900, Akira Yokosawa wrote:
> > On 2016/05/19 12:46:51 -0700, Paul E. McKenney wrote:
> >> On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
> >>> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
> >>> From: Akira Yokosawa <akiyks@gmail.com>
> >>> Date: Thu, 19 May 2016 23:34:27 +0900
> >>> Subject: [PATCH] Improve messages regarding font installation
> >>>
> >>> In commit d8318d8db866 ("Add font installation check"), there was
> >>> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
> >>> "balloon"").
> >>> However, it breaks adjustment of "##" characters in the warning
> >>> messages.
> >>> This commit fixes them.
> >>> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
> >>> a reference to item 1 of FAQ.txt.
> >>> It also adds a comment in Figure 1.1 that refers to item 1 in
> >>> FAQ.txt
> >>>
> >>> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> >>
> >> Queued, thank you!
> >>
> >> BTW, I am still getting this:
> >>
> >> #######################################################################
> >> ## Steel City Comic font is not found in the resulting PDF!          ##
> >> ## Some speech balloons in the cartoons have been rendered awkwardly.##
> >> ## See item 1 in FAQ-BUILD.txt for the solution.                     ##
> >> #######################################################################
> >>
> >> I do have some fonts installed:
> >>
> >> $ fc-list | grep steel
> >> /usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta
> >>
> >> This is from steel-city-comic.regular.ttf.  Is there something more that
> >> I need?  (I do plead much ignorance on fonts.)
> > 
> > I didn't expect you to hit this...
> > 
> > You mean the characters in Figures 3.1 and 3.2 don't fit in the balloons?

They look fine.  It is just that I get the error message above every
time I do "make".

> > "make touchsvg; make" should help you, but I think you've already tried it.

I did that.

> > "make neatfreak; make" will rebuild all the pdf files, but I don't expect
> > any difference in the result.

And I did that, too.

> > Could you send me "cartoons/r-2014-Old-man-and-Brat.pdf" off the list?

Sent separately.

> >                                                     Thanks, Akira
> 
> Or do you happen to have some environment variable set to make grep ignore
> binary files, don't you?

Not that I know of.  And "strings perfbook.pdf | grep -i steel" gives
me no output as well.  Ditto for cartoons/r-2014-Old-man-and-Brat.pdf.

> Then the grep in runlatex.sh needs to override it. I'll see what can be done.

Color me confused.

							Thanx, Paul


> >>> ---
> >>>  FAQ-BUILD.txt         | 6 +++---
> >>>  howto/howto.tex       | 9 +++++----
> >>>  utilities/eps2pdf.sh  | 2 +-
> >>>  utilities/runlatex.sh | 2 +-
> >>>  4 files changed, 10 insertions(+), 9 deletions(-)
> >>>
> >>> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
> >>> index d963534..e888725 100644
> >>> --- a/FAQ-BUILD.txt
> >>> +++ b/FAQ-BUILD.txt
> >>> @@ -1,6 +1,6 @@
> >>> -1.	I have installed the Steel City Comic font, and rebuilt
> >>> -	"perfbook.pdf".  But it doesn't seem to have any effect.
> >>> -	What am I missing?
> >>> +1.	I have installed the Steel City Comic font referring to item 1
> >>> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
> >>> +	have any effect. What am I missing?
> >>>
> >>>  	A.	If you built "perfbook.pdf" before installing the font,
> >>>  		you need to rebuild cartoons whose sources are .svg
> >>> diff --git a/howto/howto.tex b/howto/howto.tex
> >>> index 6980114..7a70bb4 100644
> >>> --- a/howto/howto.tex
> >>> +++ b/howto/howto.tex
> >>> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
> >>>  \begin{verbatim}
> >>>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
> >>>    2 cd perfbook
> >>> -  3 make
> >>> -  4 evince perfbook.pdf & # Two-column version
> >>> -  5 make perfbook-1c.pdf
> >>> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
> >>> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
> >>> +  4 make
> >>> +  5 evince perfbook.pdf & # Two-column version
> >>> +  6 make perfbook-1c.pdf
> >>> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
> >>>  \end{verbatim}
> >>>  }
> >>>  \caption{Creating an Up-To-Date PDF}
> >>> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
> >>> index 1497bdd..0a82d57 100644
> >>> --- a/utilities/eps2pdf.sh
> >>> +++ b/utilities/eps2pdf.sh
> >>> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
> >>>  then
> >>>  	echo "#######################################################################"
> >>>  	echo "## Steel City Comic font is not found in the font cache!             ##"
> >>> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
> >>> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
> >>>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
> >>>  	echo "## Nevertheless, this build will resume in a short while.            ##"
> >>>  	echo "#######################################################################"
> >>> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
> >>> index 9ae9021..94d46d6 100644
> >>> --- a/utilities/runlatex.sh
> >>> +++ b/utilities/runlatex.sh
> >>> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
> >>>  then
> >>>  	echo "#######################################################################"
> >>>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
> >>> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
> >>> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
> >>>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
> >>>  	echo "#######################################################################"
> >>>  fi
> >>> -- 
> >>> 1.9.1
> >>>
> >>
> >>
> > 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-20  4:13         ` Paul E. McKenney
@ 2016-05-20 12:00           ` Akira Yokosawa
  2016-05-20 12:01             ` [PATCH] runlatex.sh: Band-aide fix of false positive in font check Akira Yokosawa
  2016-05-20 23:43             ` [PATCH] Improve messages regarding font installation Akira Yokosawa
  0 siblings, 2 replies; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-20 12:00 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

On 2016/05/19 21:13:08 -0700, Paul E. McKenney wrote:
> On Fri, May 20, 2016 at 07:42:53AM +0900, Akira Yokosawa wrote:
>> On 2016/05/20 07:18 +0900, Akira Yokosawa wrote:
>>> On 2016/05/19 12:46:51 -0700, Paul E. McKenney wrote:
>>>> On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
>>>>> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
>>>>> From: Akira Yokosawa <akiyks@gmail.com>
>>>>> Date: Thu, 19 May 2016 23:34:27 +0900
>>>>> Subject: [PATCH] Improve messages regarding font installation
>>>>>
>>>>> In commit d8318d8db866 ("Add font installation check"), there was
>>>>> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
>>>>> "balloon"").
>>>>> However, it breaks adjustment of "##" characters in the warning
>>>>> messages.
>>>>> This commit fixes them.
>>>>> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
>>>>> a reference to item 1 of FAQ.txt.
>>>>> It also adds a comment in Figure 1.1 that refers to item 1 in
>>>>> FAQ.txt
>>>>>
>>>>> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
>>>>
>>>> Queued, thank you!
>>>>
>>>> BTW, I am still getting this:
>>>>
>>>> #######################################################################
>>>> ## Steel City Comic font is not found in the resulting PDF!          ##
>>>> ## Some speech balloons in the cartoons have been rendered awkwardly.##
>>>> ## See item 1 in FAQ-BUILD.txt for the solution.                     ##
>>>> #######################################################################
>>>>
>>>> I do have some fonts installed:
>>>>
>>>> $ fc-list | grep steel
>>>> /usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta
>>>>
>>>> This is from steel-city-comic.regular.ttf.  Is there something more that
>>>> I need?  (I do plead much ignorance on fonts.)
>>>
>>> I didn't expect you to hit this...
>>>
>>> You mean the characters in Figures 3.1 and 3.2 don't fit in the balloons?
> 
> They look fine.  It is just that I get the error message above every
> time I do "make".
> 
>>> "make touchsvg; make" should help you, but I think you've already tried it.
> 
> I did that.
> 
>>> "make neatfreak; make" will rebuild all the pdf files, but I don't expect
>>> any difference in the result.
> 
> And I did that, too.
> 
>>> Could you send me "cartoons/r-2014-Old-man-and-Brat.pdf" off the list?
> 
> Sent separately.
> 
>>>                                                     Thanks, Akira
>>
>> Or do you happen to have some environment variable set to make grep ignore
>> binary files, don't you?
> 
> Not that I know of.  And "strings perfbook.pdf | grep -i steel" gives
> me no output as well.  Ditto for cartoons/r-2014-Old-man-and-Brat.pdf.
> 
>> Then the grep in runlatex.sh needs to override it. I'll see what can be done.
> 
> Color me confused.

It seems cairo version you are using is 1.10.2.
The version I'm using is 1.13.1.

As a band-aid fix, I'll send a patch in reply to this message.
Please note that there may be conflict with the previous patch
("Improve messages regarding font installation").

                                                     Thanks, Akira

> 
> 							Thanx, Paul
> 
> 
>>>>> ---
>>>>>  FAQ-BUILD.txt         | 6 +++---
>>>>>  howto/howto.tex       | 9 +++++----
>>>>>  utilities/eps2pdf.sh  | 2 +-
>>>>>  utilities/runlatex.sh | 2 +-
>>>>>  4 files changed, 10 insertions(+), 9 deletions(-)
>>>>>
>>>>> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
>>>>> index d963534..e888725 100644
>>>>> --- a/FAQ-BUILD.txt
>>>>> +++ b/FAQ-BUILD.txt
>>>>> @@ -1,6 +1,6 @@
>>>>> -1.	I have installed the Steel City Comic font, and rebuilt
>>>>> -	"perfbook.pdf".  But it doesn't seem to have any effect.
>>>>> -	What am I missing?
>>>>> +1.	I have installed the Steel City Comic font referring to item 1
>>>>> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
>>>>> +	have any effect. What am I missing?
>>>>>
>>>>>  	A.	If you built "perfbook.pdf" before installing the font,
>>>>>  		you need to rebuild cartoons whose sources are .svg
>>>>> diff --git a/howto/howto.tex b/howto/howto.tex
>>>>> index 6980114..7a70bb4 100644
>>>>> --- a/howto/howto.tex
>>>>> +++ b/howto/howto.tex
>>>>> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
>>>>>  \begin{verbatim}
>>>>>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
>>>>>    2 cd perfbook
>>>>> -  3 make
>>>>> -  4 evince perfbook.pdf & # Two-column version
>>>>> -  5 make perfbook-1c.pdf
>>>>> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
>>>>> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
>>>>> +  4 make
>>>>> +  5 evince perfbook.pdf & # Two-column version
>>>>> +  6 make perfbook-1c.pdf
>>>>> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
>>>>>  \end{verbatim}
>>>>>  }
>>>>>  \caption{Creating an Up-To-Date PDF}
>>>>> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
>>>>> index 1497bdd..0a82d57 100644
>>>>> --- a/utilities/eps2pdf.sh
>>>>> +++ b/utilities/eps2pdf.sh
>>>>> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
>>>>>  then
>>>>>  	echo "#######################################################################"
>>>>>  	echo "## Steel City Comic font is not found in the font cache!             ##"
>>>>> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
>>>>> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
>>>>>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
>>>>>  	echo "## Nevertheless, this build will resume in a short while.            ##"
>>>>>  	echo "#######################################################################"
>>>>> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
>>>>> index 9ae9021..94d46d6 100644
>>>>> --- a/utilities/runlatex.sh
>>>>> +++ b/utilities/runlatex.sh
>>>>> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
>>>>>  then
>>>>>  	echo "#######################################################################"
>>>>>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
>>>>> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
>>>>> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
>>>>>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
>>>>>  	echo "#######################################################################"
>>>>>  fi
>>>>> -- 
>>>>> 1.9.1
>>>>>
>>>>
>>>>
>>>
>>
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH] runlatex.sh: Band-aide fix of false positive in font check
  2016-05-20 12:00           ` Akira Yokosawa
@ 2016-05-20 12:01             ` Akira Yokosawa
  2016-05-20 23:43             ` [PATCH] Improve messages regarding font installation Akira Yokosawa
  1 sibling, 0 replies; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-20 12:01 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

From 847855f817eaf1778d0c6a4bc6fb58f9541ba6f0 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 20 May 2016 20:45:14 +0900
Subject: [PATCH] runlatex.sh: Band-aide fix of false positive in font check

inkscape's font embedding seems to depend on the version of cairo.
This commit restricts the check in runlatex.sh to a specific
version of cairo (1.13.1).
When more info is available on the version range we can check in
this way, the restriction can be loosened.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 utilities/runlatex.sh | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
index 9ae9021..0be1229 100644
--- a/utilities/runlatex.sh
+++ b/utilities/runlatex.sh
@@ -113,12 +113,15 @@ echo "'$basename.pdf' is ready."
 # to avoid redundant run of bibtex and pdflatex
 touch $basename.bbl
 touch $basename.pdf
-if ! grep -q -i "steel city comic" cartoons/*.pdf
+if strings cartoons/r-2014-Old-man-and-Brat.pdf | grep -q -i "cairo 1.13.1"
 then
-	echo "#######################################################################"
-	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
-	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
-	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
-	echo "#######################################################################"
+	if ! strings cartoons/r-2014-Old-man-and-Brat.pdf | grep -q -i "steel city comic"
+	then
+		echo "#######################################################################"
+		echo "## Steel City Comic font is not found in the resulting PDF!          ##"
+		echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
+		echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
+		echo "#######################################################################"
+	fi
 fi
 exit 0
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH] Improve messages regarding font installation
  2016-05-20 12:00           ` Akira Yokosawa
  2016-05-20 12:01             ` [PATCH] runlatex.sh: Band-aide fix of false positive in font check Akira Yokosawa
@ 2016-05-20 23:43             ` Akira Yokosawa
  2016-05-21 10:22               ` Test result on Ubuntu 16.04 (was Re: [PATCH] Improve messages regarding font installation) Akira Yokosawa
  1 sibling, 1 reply; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-20 23:43 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

On 2016/05/20 21:00 +0900, Akira Yokosawa wrote:
> On 2016/05/19 21:13:08 -0700, Paul E. McKenney wrote:
>> On Fri, May 20, 2016 at 07:42:53AM +0900, Akira Yokosawa wrote:
>>> On 2016/05/20 07:18 +0900, Akira Yokosawa wrote:
>>>> On 2016/05/19 12:46:51 -0700, Paul E. McKenney wrote:
>>>>> On Fri, May 20, 2016 at 12:06:58AM +0900, Akira Yokosawa wrote:
>>>>>> >From 42dc2a0371d48c0b23f4ed70d556ef15213b5f69 Mon Sep 17 00:00:00 2001
>>>>>> From: Akira Yokosawa <akiyks@gmail.com>
>>>>>> Date: Thu, 19 May 2016 23:34:27 +0900
>>>>>> Subject: [PATCH] Improve messages regarding font installation
>>>>>>
>>>>>> In commit d8318d8db866 ("Add font installation check"), there was
>>>>>> typos already fixed by commit adc5a0ed9702 ("Fix typo "baloon" ->
>>>>>> "balloon"").
>>>>>> However, it breaks adjustment of "##" characters in the warning
>>>>>> messages.
>>>>>> This commit fixes them.
>>>>>> It also improves explanation of item 1 in FAQ-BUILD.txt by adding
>>>>>> a reference to item 1 of FAQ.txt.
>>>>>> It also adds a comment in Figure 1.1 that refers to item 1 in
>>>>>> FAQ.txt
>>>>>>
>>>>>> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
>>>>>
>>>>> Queued, thank you!
>>>>>
>>>>> BTW, I am still getting this:
>>>>>
>>>>> #######################################################################
>>>>> ## Steel City Comic font is not found in the resulting PDF!          ##
>>>>> ## Some speech balloons in the cartoons have been rendered awkwardly.##
>>>>> ## See item 1 in FAQ-BUILD.txt for the solution.                     ##
>>>>> #######################################################################
>>>>>
>>>>> I do have some fonts installed:
>>>>>
>>>>> $ fc-list | grep steel
>>>>> /usr/share/fonts/truetype/steel-city-comic.regular.ttf: Steel City Comic:style=Regular,Normal,obyčejné,Standard,Κανονικά,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,Arrunta
>>>>>
>>>>> This is from steel-city-comic.regular.ttf.  Is there something more that
>>>>> I need?  (I do plead much ignorance on fonts.)
>>>>
>>>> I didn't expect you to hit this...
>>>>
>>>> You mean the characters in Figures 3.1 and 3.2 don't fit in the balloons?
>>
>> They look fine.  It is just that I get the error message above every
>> time I do "make".
>>
>>>> "make touchsvg; make" should help you, but I think you've already tried it.
>>
>> I did that.
>>
>>>> "make neatfreak; make" will rebuild all the pdf files, but I don't expect
>>>> any difference in the result.
>>
>> And I did that, too.
>>
>>>> Could you send me "cartoons/r-2014-Old-man-and-Brat.pdf" off the list?
>>
>> Sent separately.
>>
>>>>                                                     Thanks, Akira
>>>
>>> Or do you happen to have some environment variable set to make grep ignore
>>> binary files, don't you?
>>
>> Not that I know of.  And "strings perfbook.pdf | grep -i steel" gives
>> me no output as well.  Ditto for cartoons/r-2014-Old-man-and-Brat.pdf.
>>
>>> Then the grep in runlatex.sh needs to override it. I'll see what can be done.
>>
>> Color me confused.
> 
> It seems cairo version you are using is 1.10.2.
> The version I'm using is 1.13.1.
> 
> As a band-aid fix, I'll send a patch in reply to this message.
> Please note that there may be conflict with the previous patch
> ("Improve messages regarding font installation").
> 

Hi, Paul.

I've made a better patch series relative to current master.
I tested on both Ubuntu 12.04 and Ubuntu 14.04.
I don't have Ubuntu 16.04 yet, but I'm going to test on it.
I'll send a pull request later.
(Yes, I'm using "git rebase -i")

                                                  Thanks, Akira

>                                                      Thanks, Akira
> 
>>
>> 							Thanx, Paul
>>
>>
>>>>>> ---
>>>>>>  FAQ-BUILD.txt         | 6 +++---
>>>>>>  howto/howto.tex       | 9 +++++----
>>>>>>  utilities/eps2pdf.sh  | 2 +-
>>>>>>  utilities/runlatex.sh | 2 +-
>>>>>>  4 files changed, 10 insertions(+), 9 deletions(-)
>>>>>>
>>>>>> diff --git a/FAQ-BUILD.txt b/FAQ-BUILD.txt
>>>>>> index d963534..e888725 100644
>>>>>> --- a/FAQ-BUILD.txt
>>>>>> +++ b/FAQ-BUILD.txt
>>>>>> @@ -1,6 +1,6 @@
>>>>>> -1.	I have installed the Steel City Comic font, and rebuilt
>>>>>> -	"perfbook.pdf".  But it doesn't seem to have any effect.
>>>>>> -	What am I missing?
>>>>>> +1.	I have installed the Steel City Comic font referring to item 1
>>>>>> +	in FAQ.txt, and rebuilt "perfbook.pdf". But it doesn't seem to
>>>>>> +	have any effect. What am I missing?
>>>>>>
>>>>>>  	A.	If you built "perfbook.pdf" before installing the font,
>>>>>>  		you need to rebuild cartoons whose sources are .svg
>>>>>> diff --git a/howto/howto.tex b/howto/howto.tex
>>>>>> index 6980114..7a70bb4 100644
>>>>>> --- a/howto/howto.tex
>>>>>> +++ b/howto/howto.tex
>>>>>> @@ -354,10 +354,11 @@ Other types of systems have well-known ways of locating files by filename.
>>>>>>  \begin{verbatim}
>>>>>>    1 git clone git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/perfbook.git
>>>>>>    2 cd perfbook
>>>>>> -  3 make
>>>>>> -  4 evince perfbook.pdf & # Two-column version
>>>>>> -  5 make perfbook-1c.pdf
>>>>>> -  6 evince perfbook-1c.pdf & # One-column version for e-readers
>>>>>> +  3 # You may need to install a font here. See item 1 in FAQ.txt.
>>>>>> +  4 make
>>>>>> +  5 evince perfbook.pdf & # Two-column version
>>>>>> +  6 make perfbook-1c.pdf
>>>>>> +  7 evince perfbook-1c.pdf & # One-column version for e-readers
>>>>>>  \end{verbatim}
>>>>>>  }
>>>>>>  \caption{Creating an Up-To-Date PDF}
>>>>>> diff --git a/utilities/eps2pdf.sh b/utilities/eps2pdf.sh
>>>>>> index 1497bdd..0a82d57 100644
>>>>>> --- a/utilities/eps2pdf.sh
>>>>>> +++ b/utilities/eps2pdf.sh
>>>>>> @@ -24,7 +24,7 @@ if ! fc-list | grep -q steel
>>>>>>  then
>>>>>>  	echo "#######################################################################"
>>>>>>  	echo "## Steel City Comic font is not found in the font cache!             ##"
>>>>>> -	echo "## Some speech balloons in the cartoons would be rendered awkwardly.  ##"
>>>>>> +	echo "## Some speech balloons in the cartoons would be rendered awkwardly. ##"
>>>>>>  	echo "## See item 1 in FAQ.txt for how to install the font.                ##"
>>>>>>  	echo "## Nevertheless, this build will resume in a short while.            ##"
>>>>>>  	echo "#######################################################################"
>>>>>> diff --git a/utilities/runlatex.sh b/utilities/runlatex.sh
>>>>>> index 9ae9021..94d46d6 100644
>>>>>> --- a/utilities/runlatex.sh
>>>>>> +++ b/utilities/runlatex.sh
>>>>>> @@ -117,7 +117,7 @@ if ! grep -q -i "steel city comic" cartoons/*.pdf
>>>>>>  then
>>>>>>  	echo "#######################################################################"
>>>>>>  	echo "## Steel City Comic font is not found in the resulting PDF!          ##"
>>>>>> -	echo "## Some speech balloons in the cartoons have been rendered awkwardly. ##"
>>>>>> +	echo "## Some speech balloons in the cartoons have been rendered awkwardly.##"
>>>>>>  	echo "## See item 1 in FAQ-BUILD.txt for the solution.                     ##"
>>>>>>  	echo "#######################################################################"
>>>>>>  fi
>>>>>> -- 
>>>>>> 1.9.1
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>>
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Test result on Ubuntu 16.04 (was Re: [PATCH] Improve messages regarding font installation)
  2016-05-20 23:43             ` [PATCH] Improve messages regarding font installation Akira Yokosawa
@ 2016-05-21 10:22               ` Akira Yokosawa
  0 siblings, 0 replies; 10+ messages in thread
From: Akira Yokosawa @ 2016-05-21 10:22 UTC (permalink / raw)
  To: paulmck; +Cc: perfbook, Akira Yokosawa

On 2016/05/21 8:43 +0900, Akira Yokosawa wrote:
> On 2016/05/20 21:00 +0900, Akira Yokosawa wrote:
[snip]
>>
>> It seems cairo version you are using is 1.10.2.
>> The version I'm using is 1.13.1.
>>
>> As a band-aid fix, I'll send a patch in reply to this message.
>> Please note that there may be conflict with the previous patch
>> ("Improve messages regarding font installation").
>>
> 
> Hi, Paul.
> 
> I've made a better patch series relative to current master.
> I tested on both Ubuntu 12.04 and Ubuntu 14.04.
> I don't have Ubuntu 16.04 yet, but I'm going to test on it.

Hi, Paul,
I tested the patch on Ubuntu 16.04.
It works fine there.

FYI, there is a minor problem in eps2pdf.sh with a2ping spitting
a lot of messages
"GPL Ghostscript 9.18: Unrecoverable error, exit code 1".

Luckily for us, the resulting PDF looks fine to me.
Googling the above message hits a lot of pages.
But I can't figure out a workaround yet.

                                               Thanks, Akira

> I'll send a pull request later.
> (Yes, I'm using "git rebase -i")
> 
>                                                   Thanks, Akira
> 
[snip]


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2016-05-21 10:22 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-19 15:01 Fix typo "baloon" -> "balloon" Akira Yokosawa
2016-05-19 15:06 ` [PATCH] Improve messages regarding font installation Akira Yokosawa
2016-05-19 19:46   ` Paul E. McKenney
2016-05-19 22:18     ` Akira Yokosawa
2016-05-19 22:42       ` Akira Yokosawa
2016-05-20  4:13         ` Paul E. McKenney
2016-05-20 12:00           ` Akira Yokosawa
2016-05-20 12:01             ` [PATCH] runlatex.sh: Band-aide fix of false positive in font check Akira Yokosawa
2016-05-20 23:43             ` [PATCH] Improve messages regarding font installation Akira Yokosawa
2016-05-21 10:22               ` Test result on Ubuntu 16.04 (was Re: [PATCH] Improve messages regarding font installation) Akira Yokosawa

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.