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
=== modified file '__init__.py'
--- __init__.py 2009-07-15 17:06:04 +0000
+++ __init__.py 2009-07-17 18:15:54 +0000
@@ -29,6 +29,10 @@
29from bzrlib.commands import plugin_cmds29from bzrlib.commands import plugin_cmds
30from bzrlib.directory_service import directories30from bzrlib.directory_service import directories
3131
32from setup import (
33 bzr_plugin_version as version_info,
34 )
35
32commands = {36commands = {
33 "test_builddeb": [],37 "test_builddeb": [],
34 "builddeb": ["bd"],38 "builddeb": ["bd"],
3539
=== modified file 'setup.py'
--- setup.py 2009-03-05 16:22:04 +0000
+++ setup.py 2009-07-17 18:15:54 +0000
@@ -20,18 +20,36 @@
20# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA20# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
21#21#
2222
23from distutils.core import setup23bzr_plugin_name = 'builddeb'
2424
25setup(name="bzr-builddeb",25bzr_plugin_version = (2, 1, 1, 'dev', 0)
26 version="2.1.1",26
27 description="Build a .deb from a bzr branch",27bzr_commands = [
28 author="James Westby",28 "test_builddeb",
29 author_email="jw+debian@jameswestby.net",29 "builddeb",
30 license = "GNU GPL v2",30 "merge_upstream",
31 url="http://jameswestby.net/bzr/bzr-builddeb/",31 "import_dsc",
32 packages=['bzrlib.plugins.builddeb',32 "bd_do",
33 'bzrlib.plugins.builddeb.tests',33 "mark_uploaded",
34 'bzrlib.plugins.builddeb.tests.blackbox'],34 ]
35 package_dir={'bzrlib.plugins.builddeb': '.'},35
36 scripts=['bzr-buildpackage'],36
37 data_files=[('share/man/man1', ['bzr-buildpackage.1'])])37if __name__ == '__main__':
38
39 from distutils.core import setup
40
41 version_string = ".".join([str(v) for v in bzr_plugin_version[:3]])
42
43 setup(name="bzr-builddeb",
44 version=version_string,
45 description="Build a .deb from a bzr branch",
46 author="James Westby",
47 author_email="jw+debian@jameswestby.net",
48 license = "GNU GPL v2",
49 url="http://jameswestby.net/bzr/bzr-builddeb/",
50 packages=['bzrlib.plugins.builddeb',
51 'bzrlib.plugins.builddeb.tests',
52 'bzrlib.plugins.builddeb.tests.blackbox'],
53 package_dir={'bzrlib.plugins.builddeb': '.'},
54 scripts=['bzr-buildpackage'],
55 data_files=[('share/man/man1', ['bzr-buildpackage.1'])])

Subscribers

People subscribed via source and target branches