Merge lp:~jelmer/bzr-builddeb/plugin-info into lp:~bzr-builddeb-hackers/bzr-builddeb/trunk-old

Proposed by Jelmer Vernooij
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jelmer/bzr-builddeb/plugin-info
Merge into: lp:~bzr-builddeb-hackers/bzr-builddeb/trunk-old
Diff against target: None lines
To merge this branch: bzr merge lp:~jelmer/bzr-builddeb/plugin-info

This proposal supersedes a proposal from 2009-07-17.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) wrote : Posted in a previous version of this proposal

This branch adds some basic plugin information as suggested by plugin-api.txt. It also adds the "version_info" attribute to __init__.py, so "bzr plugins" will print the plugin version.

Revision history for this message
James Westby (james-w) wrote : Posted in a previous version of this proposal

> This branch adds some basic plugin information as suggested by plugin-api.txt.
> It also adds the "version_info" attribute to __init__.py, so "bzr plugins"
> will print the plugin version.

This will make the plugin fail to import when packaged won't it?

I like the aim, but organising the information such that the setup.py
isn't required for the plugin to work would be a better solution
in my opinion.

Thanks,

James

review: Needs Information

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '__init__.py'
2--- __init__.py 2009-07-15 17:06:04 +0000
3+++ __init__.py 2009-07-17 18:15:54 +0000
4@@ -29,6 +29,10 @@
5 from bzrlib.commands import plugin_cmds
6 from bzrlib.directory_service import directories
7
8+from setup import (
9+ bzr_plugin_version as version_info,
10+ )
11+
12 commands = {
13 "test_builddeb": [],
14 "builddeb": ["bd"],
15
16=== modified file 'setup.py'
17--- setup.py 2009-03-05 16:22:04 +0000
18+++ setup.py 2009-07-17 18:15:54 +0000
19@@ -20,18 +20,36 @@
20 # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
21 #
22
23-from distutils.core import setup
24-
25-setup(name="bzr-builddeb",
26- version="2.1.1",
27- description="Build a .deb from a bzr branch",
28- author="James Westby",
29- author_email="jw+debian@jameswestby.net",
30- license = "GNU GPL v2",
31- url="http://jameswestby.net/bzr/bzr-builddeb/",
32- packages=['bzrlib.plugins.builddeb',
33- 'bzrlib.plugins.builddeb.tests',
34- 'bzrlib.plugins.builddeb.tests.blackbox'],
35- package_dir={'bzrlib.plugins.builddeb': '.'},
36- scripts=['bzr-buildpackage'],
37- data_files=[('share/man/man1', ['bzr-buildpackage.1'])])
38+bzr_plugin_name = 'builddeb'
39+
40+bzr_plugin_version = (2, 1, 1, 'dev', 0)
41+
42+bzr_commands = [
43+ "test_builddeb",
44+ "builddeb",
45+ "merge_upstream",
46+ "import_dsc",
47+ "bd_do",
48+ "mark_uploaded",
49+ ]
50+
51+
52+if __name__ == '__main__':
53+
54+ from distutils.core import setup
55+
56+ version_string = ".".join([str(v) for v in bzr_plugin_version[:3]])
57+
58+ setup(name="bzr-builddeb",
59+ version=version_string,
60+ description="Build a .deb from a bzr branch",
61+ author="James Westby",
62+ author_email="jw+debian@jameswestby.net",
63+ license = "GNU GPL v2",
64+ url="http://jameswestby.net/bzr/bzr-builddeb/",
65+ packages=['bzrlib.plugins.builddeb',
66+ 'bzrlib.plugins.builddeb.tests',
67+ 'bzrlib.plugins.builddeb.tests.blackbox'],
68+ package_dir={'bzrlib.plugins.builddeb': '.'},
69+ scripts=['bzr-buildpackage'],
70+ data_files=[('share/man/man1', ['bzr-buildpackage.1'])])

Subscribers

People subscribed via source and target branches