Merge lp:~shkn/bake/remove-pkg-config-autogeneration into lp:bake

Proposed by shuerhaaken
Status: Merged
Merged at revision: 476
Proposed branch: lp:~shkn/bake/remove-pkg-config-autogeneration
Merge into: lp:bake
Diff against target: 150 lines (+0/-34)
6 files modified
src/module-vala.vala (+0/-17)
tests/vala-library-complex/expected (+0/-4)
tests/vala-library-install-directory/expected (+0/-4)
tests/vala-library-no-install/expected (+0/-1)
tests/vala-library-override-names/expected (+0/-4)
tests/vala-library-simple/expected (+0/-4)
To merge this branch: bzr merge lp:~shkn/bake/remove-pkg-config-autogeneration
Reviewer Review Type Date Requested Status
Robert Ancell Approve
Review via email: mp+157527@code.launchpad.net

Description of the change

Fixes Bug #1162704 - Remove pkg-config autogeneration

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Thanks, I committed this along with a similar commit to remove the autogeneration from the C/C++ library support. Note if you do:
bzr commit -m "Remove pkg-config autogeneration" --fixes lp:1162704
it will automatically link this commit to the bug.

review: Approve
Revision history for this message
shuerhaaken (shkn) wrote :

Ok, next time with --fixes arg.

I'd be glad if you could use --author arg for my contributions next time ;)

I'm sorry I overlooked the pkg-config stuff in gcc module.

Revision history for this message
Robert Ancell (robert-ancell) wrote :

Ah, sorry. I thought bzr merge would just set me as the merge contributor, but checking with bzr blame shows it considers the merged branches to be my work. I'll set author from now on.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/module-vala.vala'
--- src/module-vala.vala 2013-04-05 23:48:29 +0000
+++ src/module-vala.vala 2013-04-06 21:40:26 +0000
@@ -48,27 +48,10 @@
48 }48 }
4949
50 /* Generate pkg-config file */50 /* Generate pkg-config file */
51 var filename = "%s-%s.pc".printf (library.name, major_version);
52 var name = library.name;51 var name = library.name;
53 var description = library.get_variable ("description", "");
54 var requires = library.get_variable ("requires", "");
5552
56 var include_directory = Path.build_filename (recipe.include_directory, "%s-%s".printf (library.name, major_version));53 var include_directory = Path.build_filename (recipe.include_directory, "%s-%s".printf (library.name, major_version));
5754
58 rule = recipe.add_rule ();
59 recipe.build_rule.add_input (filename);
60 rule.add_output (filename);
61 rule.add_status_command ("PKG-CONFIG %s".printf (filename));
62 rule.add_command ("@echo \"Name: %s\" > %s".printf (name, filename));
63 rule.add_command ("@echo \"Description: %s\" >> %s".printf (description, filename));
64 rule.add_command ("@echo \"Version: %s\" >> %s".printf (version, filename));
65 rule.add_command ("@echo \"Requires: %s\" >> %s".printf (requires, filename));
66 rule.add_command ("@echo \"Libs: -L%s -l%s\" >> %s".printf (library.install_directory, library.name, filename));
67 rule.add_command ("@echo \"Cflags: -I%s\" >> %s".printf (include_directory, filename));
68
69 if (library.install)
70 recipe.add_install_rule (filename, Path.build_filename (library.install_directory, "pkgconfig"));
71
72 var h_filename = library.get_variable ("vala-header-name");55 var h_filename = library.get_variable ("vala-header-name");
73 if (h_filename == null)56 if (h_filename == null)
74 h_filename = "%s.h".printf (name);57 h_filename = "%s.h".printf (name);
7558
=== modified file 'tests/vala-library-complex/expected'
--- tests/vala-library-complex/expected 2013-04-05 23:56:49 +0000
+++ tests/vala-library-complex/expected 2013-04-06 21:40:26 +0000
@@ -28,8 +28,6 @@
28cp libtest.so /usr/lib/libtest.so28cp libtest.so /usr/lib/libtest.so
29cp libtest.so.0 /usr/lib/libtest.so.029cp libtest.so.0 /usr/lib/libtest.so.0
30cp libtest.a /usr/lib/libtest.a30cp libtest.a /usr/lib/libtest.a
31mkdir -p /usr/lib/pkgconfig
32cp test-0.pc /usr/lib/pkgconfig/test-0.pc
33mkdir -p /usr/include/test-031mkdir -p /usr/include/test-0
34cp test.h /usr/include/test-0/test.h32cp test.h /usr/include/test-0/test.h
35mkdir -p /usr/share/vala/vapi33mkdir -p /usr/share/vala/vapi
@@ -39,7 +37,6 @@
39rm -f /usr/lib/libtest.so37rm -f /usr/lib/libtest.so
40rm -f /usr/lib/libtest.so.038rm -f /usr/lib/libtest.so.0
41rm -f /usr/lib/libtest.a39rm -f /usr/lib/libtest.a
42rm -f /usr/lib/pkgconfig/test-0.pc
43rm -f /usr/include/test-0/test.h40rm -f /usr/include/test-0/test.h
44rm -f /usr/share/vala/vapi/test-0.vapi41rm -f /usr/share/vala/vapi/test-0.vapi
45(exit SUCCESS)42(exit SUCCESS)
@@ -50,6 +47,5 @@
50rm -f test-0.vapi47rm -f test-0.vapi
51rm -rf .built/48rm -rf .built/
52rm -f libtest.so49rm -f libtest.so
53rm -f test-0.pc
54rm -f test.pot50rm -f test.pot
55(exit SUCCESS)51(exit SUCCESS)
5652
=== modified file 'tests/vala-library-install-directory/expected'
--- tests/vala-library-install-directory/expected 2013-04-05 23:35:39 +0000
+++ tests/vala-library-install-directory/expected 2013-04-06 21:40:26 +0000
@@ -14,8 +14,6 @@
14cp libtest.so /usr/lib/test/libtest.so14cp libtest.so /usr/lib/test/libtest.so
15cp libtest.so.0 /usr/lib/test/libtest.so.015cp libtest.so.0 /usr/lib/test/libtest.so.0
16cp libtest.a /usr/lib/test/libtest.a16cp libtest.a /usr/lib/test/libtest.a
17mkdir -p /usr/lib/test/pkgconfig
18cp test-0.pc /usr/lib/test/pkgconfig/test-0.pc
19mkdir -p /usr/include/test-017mkdir -p /usr/include/test-0
20cp test.h /usr/include/test-0/test.h18cp test.h /usr/include/test-0/test.h
21mkdir -p /usr/share/vala/vapi19mkdir -p /usr/share/vala/vapi
@@ -25,7 +23,6 @@
25rm -f /usr/lib/test/libtest.so23rm -f /usr/lib/test/libtest.so
26rm -f /usr/lib/test/libtest.so.024rm -f /usr/lib/test/libtest.so.0
27rm -f /usr/lib/test/libtest.a25rm -f /usr/lib/test/libtest.a
28rm -f /usr/lib/test/pkgconfig/test-0.pc
29rm -f /usr/include/test-0/test.h26rm -f /usr/include/test-0/test.h
30rm -f /usr/share/vala/vapi/test-0.vapi27rm -f /usr/share/vala/vapi/test-0.vapi
31(exit SUCCESS)28(exit SUCCESS)
@@ -36,5 +33,4 @@
36rm -f test-0.vapi33rm -f test-0.vapi
37rm -rf .built/34rm -rf .built/
38rm -f libtest.so35rm -f libtest.so
39rm -f test-0.pc
40(exit SUCCESS)36(exit SUCCESS)
4137
=== modified file 'tests/vala-library-no-install/expected'
--- tests/vala-library-no-install/expected 2013-04-05 23:35:39 +0000
+++ tests/vala-library-no-install/expected 2013-04-06 21:40:26 +0000
@@ -20,5 +20,4 @@
20rm -f test-0.vapi20rm -f test-0.vapi
21rm -rf .built/21rm -rf .built/
22rm -f libtest.so22rm -f libtest.so
23rm -f test-0.pc
24(exit SUCCESS)23(exit SUCCESS)
2524
=== modified file 'tests/vala-library-override-names/expected'
--- tests/vala-library-override-names/expected 2013-04-05 23:53:21 +0000
+++ tests/vala-library-override-names/expected 2013-04-06 21:40:26 +0000
@@ -14,8 +14,6 @@
14cp libtest.so /usr/lib/libtest.so14cp libtest.so /usr/lib/libtest.so
15cp libtest.so.0 /usr/lib/libtest.so.015cp libtest.so.0 /usr/lib/libtest.so.0
16cp libtest.a /usr/lib/libtest.a16cp libtest.a /usr/lib/libtest.a
17mkdir -p /usr/lib/pkgconfig
18cp test-0.pc /usr/lib/pkgconfig/test-0.pc
19mkdir -p /usr/include/test-017mkdir -p /usr/include/test-0
20cp foo.h /usr/include/test-0/foo.h18cp foo.h /usr/include/test-0/foo.h
21mkdir -p /usr/share/vala/vapi19mkdir -p /usr/share/vala/vapi
@@ -25,7 +23,6 @@
25rm -f /usr/lib/libtest.so23rm -f /usr/lib/libtest.so
26rm -f /usr/lib/libtest.so.024rm -f /usr/lib/libtest.so.0
27rm -f /usr/lib/libtest.a25rm -f /usr/lib/libtest.a
28rm -f /usr/lib/pkgconfig/test-0.pc
29rm -f /usr/include/test-0/foo.h26rm -f /usr/include/test-0/foo.h
30rm -f /usr/share/vala/vapi/foo.vapi27rm -f /usr/share/vala/vapi/foo.vapi
31(exit SUCCESS)28(exit SUCCESS)
@@ -36,5 +33,4 @@
36rm -f foo.vapi33rm -f foo.vapi
37rm -rf .built/34rm -rf .built/
38rm -f libtest.so35rm -f libtest.so
39rm -f test-0.pc
40(exit SUCCESS)36(exit SUCCESS)
4137
=== modified file 'tests/vala-library-simple/expected'
--- tests/vala-library-simple/expected 2013-04-05 23:35:39 +0000
+++ tests/vala-library-simple/expected 2013-04-06 21:40:26 +0000
@@ -14,8 +14,6 @@
14cp libtest.so /usr/lib/libtest.so14cp libtest.so /usr/lib/libtest.so
15cp libtest.so.0 /usr/lib/libtest.so.015cp libtest.so.0 /usr/lib/libtest.so.0
16cp libtest.a /usr/lib/libtest.a16cp libtest.a /usr/lib/libtest.a
17mkdir -p /usr/lib/pkgconfig
18cp test-0.pc /usr/lib/pkgconfig/test-0.pc
19mkdir -p /usr/include/test-017mkdir -p /usr/include/test-0
20cp test.h /usr/include/test-0/test.h18cp test.h /usr/include/test-0/test.h
21mkdir -p /usr/share/vala/vapi19mkdir -p /usr/share/vala/vapi
@@ -25,7 +23,6 @@
25rm -f /usr/lib/libtest.so23rm -f /usr/lib/libtest.so
26rm -f /usr/lib/libtest.so.024rm -f /usr/lib/libtest.so.0
27rm -f /usr/lib/libtest.a25rm -f /usr/lib/libtest.a
28rm -f /usr/lib/pkgconfig/test-0.pc
29rm -f /usr/include/test-0/test.h26rm -f /usr/include/test-0/test.h
30rm -f /usr/share/vala/vapi/test-0.vapi27rm -f /usr/share/vala/vapi/test-0.vapi
31(exit SUCCESS)28(exit SUCCESS)
@@ -36,5 +33,4 @@
36rm -f test-0.vapi33rm -f test-0.vapi
37rm -rf .built/34rm -rf .built/
38rm -f libtest.so35rm -f libtest.so
39rm -f test-0.pc
40(exit SUCCESS)36(exit SUCCESS)

Subscribers

People subscribed via source and target branches