diff mbox

doc: images: refactor makefile

Message ID 1452195848-6517-1-git-send-email-mike.holmes@linaro.org
State Accepted
Commit ad831c8da063a3150b8a3d5ef36fa03a3ff2ee28
Headers show

Commit Message

Mike Holmes Jan. 7, 2016, 7:44 p.m. UTC
Drop per target rules for generic make

Suggested-by: Anders Roxell <anders.roxell@linaro.og>
Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
---
 doc/images/Makefile.am | 67 ++++++++++----------------------------------------
 1 file changed, 13 insertions(+), 54 deletions(-)

Comments

Bill Fischofer Jan. 7, 2016, 7:47 p.m. UTC | #1
On Thu, Jan 7, 2016 at 1:44 PM, Mike Holmes <mike.holmes@linaro.org> wrote:

> Drop per target rules for generic make

>

> Suggested-by: Anders Roxell <anders.roxell@linaro.og>

> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

>


Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>



> ---

>  doc/images/Makefile.am | 67

> ++++++++++----------------------------------------

>  1 file changed, 13 insertions(+), 54 deletions(-)

>

> diff --git a/doc/images/Makefile.am b/doc/images/Makefile.am

> index 5495b69..8ab03b4 100644

> --- a/doc/images/Makefile.am

> +++ b/doc/images/Makefile.am

> @@ -1,4 +1,14 @@

> -SVG_SRCS =  atomic_queue.svg \

> +.svg.png:

> +       convert $^ $@

> +

> +.svg.eps:

> +       convert $^ $@

> +

> +.msc.png:

> +       mscgen -T png -i $^ -o $@

> +

> +SVG_SRCS = \

> +       atomic_queue.svg \

>         ordered_queue.svg \

>         parallel_queue.svg \

>         odp_components.svg \

> @@ -11,12 +21,13 @@ SVG_SRCS =  atomic_queue.svg \

>

>  SVG_TARGETS = $(SVG_SRCS:svg=png)

>  SVG_TARGETS += $(SVG_SRCS:svg=eps)

> +

>  MSG_SRCS = resource_management.msc

>  MSG_TARGETS = $(MSG_SRCS:msc=png)

>

>  EXTRA_DIST = $(SVG_SRCS) $(MSG_SRCS)

>

> -TARGETS=

> +TARGETS=$(SVG_TARGETS) $(MSG_TARGETS)

>

>  if HAVE_IMAGEMAGIC

>  TARGETS += $(SVG_TARGETS)

> @@ -30,55 +41,3 @@ all-local: $(TARGETS)

>

>  clean-local:

>         rm -f $(SVG_TARGETS) $(MSG_TARGETS)

> -

> -atomic_queue.png: atomic_queue.svg

> -       convert $< $@

> -atomic_queue.eps: atomic_queue.svg

> -       convert $< $@

> -

> -ordered_queue.png: ordered_queue.svg

> -       convert $< $@

> -parallel_queue.eps: parallel_queue.svg

> -       convert $< $@

> -

> -parallel_queue.png: parallel_queue.svg

> -       convert $< $@

> -ordered_queue.eps: ordered_queue.svg

> -       convert $< $@

> -

> -odp_components.png: odp_components.svg

> -       convert $< $@

> -odp_components.eps: odp_components.svg

> -       convert $< $@

> -

> -odp_rx_processing.png: odp_rx_processing.svg

> -       convert $< $@

> -odp_rx_processing.eps: odp_rx_processing.svg

> -       convert $< $@

> -

> -odp_scheduling.png: odp_scheduling.svg

> -       convert $< $@

> -odp_scheduling.eps: odp_scheduling.svg

> -       convert $< $@

> -

> -odp_traffic_manager.png: odp_traffic_manager.svg

> -       convert $< $@

> -odp_traffic_manager.eps: odp_traffic_manager.svg

> -       convert $< $@

> -

> -overview.png: overview.svg

> -       convert $< $@

> -overview.eps: overview.svg

> -

> -release_git.png: release_git.svg

> -       convert $< $@

> -release_git.eps: release_git.svg

> -       convert $< $@

> -

> -simple_release_git.png: simple_release_git.svg

> -       convert $< $@

> -simple_release_git.eps: simple_release_git.svg

> -       convert $< $@

> -

> -resource_management.png: resource_management.msc

> -       mscgen -T png -i $< -o $@

> --

> 2.5.0

>

> _______________________________________________

> lng-odp mailing list

> lng-odp@lists.linaro.org

> https://lists.linaro.org/mailman/listinfo/lng-odp

>
Maxim Uvarov Jan. 11, 2016, 11:56 a.m. UTC | #2
Merged,
Maxim.

On 01/07/2016 22:47, Bill Fischofer wrote:
>
>
> On Thu, Jan 7, 2016 at 1:44 PM, Mike Holmes <mike.holmes@linaro.org 
> <mailto:mike.holmes@linaro.org>> wrote:
>
>     Drop per target rules for generic make
>
>     Suggested-by: Anders Roxell <anders.roxell@linaro.og>
>     Signed-off-by: Mike Holmes <mike.holmes@linaro.org
>     <mailto:mike.holmes@linaro.org>>
>
>
> Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org 
> <mailto:bill.fischofer@linaro.org>>
>
>     ---
>      doc/images/Makefile.am | 67
>     ++++++++++----------------------------------------
>      1 file changed, 13 insertions(+), 54 deletions(-)
>
>     diff --git a/doc/images/Makefile.am b/doc/images/Makefile.am
>     index 5495b69..8ab03b4 100644
>     --- a/doc/images/Makefile.am
>     +++ b/doc/images/Makefile.am
>     @@ -1,4 +1,14 @@
>     -SVG_SRCS =  atomic_queue.svg \
>     +.svg.png:
>     +       convert $^ $@
>     +
>     +.svg.eps:
>     +       convert $^ $@
>     +
>     +.msc.png:
>     +       mscgen -T png -i $^ -o $@
>     +
>     +SVG_SRCS = \
>     +       atomic_queue.svg \
>             ordered_queue.svg \
>             parallel_queue.svg \
>             odp_components.svg \
>     @@ -11,12 +21,13 @@ SVG_SRCS =  atomic_queue.svg \
>
>      SVG_TARGETS = $(SVG_SRCS:svg=png)
>      SVG_TARGETS += $(SVG_SRCS:svg=eps)
>     +
>      MSG_SRCS = resource_management.msc
>      MSG_TARGETS = $(MSG_SRCS:msc=png)
>
>      EXTRA_DIST = $(SVG_SRCS) $(MSG_SRCS)
>
>     -TARGETS=
>     +TARGETS=$(SVG_TARGETS) $(MSG_TARGETS)
>
>      if HAVE_IMAGEMAGIC
>      TARGETS += $(SVG_TARGETS)
>     @@ -30,55 +41,3 @@ all-local: $(TARGETS)
>
>      clean-local:
>             rm -f $(SVG_TARGETS) $(MSG_TARGETS)
>     -
>     -atomic_queue.png: atomic_queue.svg
>     -       convert $< $@
>     -atomic_queue.eps: atomic_queue.svg
>     -       convert $< $@
>     -
>     -ordered_queue.png: ordered_queue.svg
>     -       convert $< $@
>     -parallel_queue.eps: parallel_queue.svg
>     -       convert $< $@
>     -
>     -parallel_queue.png: parallel_queue.svg
>     -       convert $< $@
>     -ordered_queue.eps: ordered_queue.svg
>     -       convert $< $@
>     -
>     -odp_components.png: odp_components.svg
>     -       convert $< $@
>     -odp_components.eps: odp_components.svg
>     -       convert $< $@
>     -
>     -odp_rx_processing.png: odp_rx_processing.svg
>     -       convert $< $@
>     -odp_rx_processing.eps: odp_rx_processing.svg
>     -       convert $< $@
>     -
>     -odp_scheduling.png: odp_scheduling.svg
>     -       convert $< $@
>     -odp_scheduling.eps: odp_scheduling.svg
>     -       convert $< $@
>     -
>     -odp_traffic_manager.png: odp_traffic_manager.svg
>     -       convert $< $@
>     -odp_traffic_manager.eps: odp_traffic_manager.svg
>     -       convert $< $@
>     -
>     -overview.png: overview.svg
>     -       convert $< $@
>     -overview.eps: overview.svg
>     -
>     -release_git.png: release_git.svg
>     -       convert $< $@
>     -release_git.eps: release_git.svg
>     -       convert $< $@
>     -
>     -simple_release_git.png: simple_release_git.svg
>     -       convert $< $@
>     -simple_release_git.eps: simple_release_git.svg
>     -       convert $< $@
>     -
>     -resource_management.png: resource_management.msc
>     -       mscgen -T png -i $< -o $@
>     --
>     2.5.0
>
>     _______________________________________________
>     lng-odp mailing list
>     lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>     https://lists.linaro.org/mailman/listinfo/lng-odp
>
>
>
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
diff mbox

Patch

diff --git a/doc/images/Makefile.am b/doc/images/Makefile.am
index 5495b69..8ab03b4 100644
--- a/doc/images/Makefile.am
+++ b/doc/images/Makefile.am
@@ -1,4 +1,14 @@ 
-SVG_SRCS =  atomic_queue.svg \
+.svg.png:
+	convert $^ $@
+
+.svg.eps:
+	convert $^ $@
+
+.msc.png:
+	mscgen -T png -i $^ -o $@
+
+SVG_SRCS = \
+	atomic_queue.svg \
 	ordered_queue.svg \
 	parallel_queue.svg \
 	odp_components.svg \
@@ -11,12 +21,13 @@  SVG_SRCS =  atomic_queue.svg \
 
 SVG_TARGETS = $(SVG_SRCS:svg=png)
 SVG_TARGETS += $(SVG_SRCS:svg=eps)
+
 MSG_SRCS = resource_management.msc
 MSG_TARGETS = $(MSG_SRCS:msc=png)
 
 EXTRA_DIST = $(SVG_SRCS) $(MSG_SRCS)
 
-TARGETS=
+TARGETS=$(SVG_TARGETS) $(MSG_TARGETS)
 
 if HAVE_IMAGEMAGIC
 TARGETS += $(SVG_TARGETS)
@@ -30,55 +41,3 @@  all-local: $(TARGETS)
 
 clean-local:
 	rm -f $(SVG_TARGETS) $(MSG_TARGETS)
-
-atomic_queue.png: atomic_queue.svg
-	convert $< $@
-atomic_queue.eps: atomic_queue.svg
-	convert $< $@
-
-ordered_queue.png: ordered_queue.svg
-	convert $< $@
-parallel_queue.eps: parallel_queue.svg
-	convert $< $@
-
-parallel_queue.png: parallel_queue.svg
-	convert $< $@
-ordered_queue.eps: ordered_queue.svg
-	convert $< $@
-
-odp_components.png: odp_components.svg
-	convert $< $@
-odp_components.eps: odp_components.svg
-	convert $< $@
-
-odp_rx_processing.png: odp_rx_processing.svg
-	convert $< $@
-odp_rx_processing.eps: odp_rx_processing.svg
-	convert $< $@
-
-odp_scheduling.png: odp_scheduling.svg
-	convert $< $@
-odp_scheduling.eps: odp_scheduling.svg
-	convert $< $@
-
-odp_traffic_manager.png: odp_traffic_manager.svg
-	convert $< $@
-odp_traffic_manager.eps: odp_traffic_manager.svg
-	convert $< $@
-
-overview.png: overview.svg
-	convert $< $@
-overview.eps: overview.svg
-
-release_git.png: release_git.svg
-	convert $< $@
-release_git.eps: release_git.svg
-	convert $< $@
-
-simple_release_git.png: simple_release_git.svg
-	convert $< $@
-simple_release_git.eps: simple_release_git.svg
-	convert $< $@
-
-resource_management.png: resource_management.msc
-	mscgen -T png -i $< -o $@