[ovs-dev] Patch: Add openvswitch-python package

[Ben Pfaff]

[Sent privately before; expanding distribution now.]

I agree with Romain.

Debian Python Policy says this:

   2.2 Module Package Names

   Public modules used by other packages must have their binary package
   name prefixed with python-. It is recommended to use this prefix for
   all packages with public modules as they may be used by other
   packages in the future.

On Mon, Sep 20, 2010 at 6:42 PM, Sajjad Lateef <slateef at nicira.com> wrote:
> Hi Romain,
>
> I thought about that naming scheme initially (and also creating a
> python distutils package). But, in the interest of keeping the
> openvswitch packages consistent with each other, I elected to use
> openvswitch-python.
>
> Thanks,
> Sajjad
>
> On Mon, Sep 20, 2010 at 4:54 PM, Romain Lenglet <romain at midokura.jp> wrote:
>> Hi Sajjad,
>> I would recommend naming this package 'python-openvswitch' instead of
>> 'openvswith-python', to be consistent with other Python library packages.
>>
>> On Tue, Sep 21, 2010 at 7:00 AM, Sajjad Lateef <slateef at nicira.com> wrote:
>>>
>>> On Mon, Sep 20, 2010 at 2:58 PM, Ben Pfaff <blp at nicira.com> wrote:
>>> > On Mon, Sep 20, 2010 at 02:24:35PM -0700, Sajjad Lateef wrote:
>>> >
>>> >> On my test system with openvswitch-python debian package installed:
>>> >> ~# python
>>> >> Python 2.5.2 (r252:60911, Jan 24 2010, 14:53:14)
>>> >> [GCC 4.3.2] on linux2
>>> >> Type "help", "copyright", "credits" or "license" for more information.
>>> >> >>> import ovs.json
>>> >> >>> dir(ovs.json)
>>> >> ['Parser', 'StringIO', '__builtins__', '__doc__', '__dump_string',
>>> >> '__file__', '__name__', 'escapes', 'from_file', 'from_stream',
>>> >> 'from_string', 'i', 're', 'sys', 'to_file', 'to_stream', 'to_string']
>>> >> >>> from ovs import daemon
>>> >> >>> dir(daemon)
>>> >> ['LONG_OPTIONS', '__builtins__', '__doc__', '__file__', '__name__',
>>> >> '_already_running', '_chdir', '_close_standard_fds', '_daemonize_fd',
>>> >> '_detach', '_fork_and_wait_for_startup', '_fork_notify_startup',
>>> >> '_make_pidfile', '_monitor', '_monitor_daemon', '_overwrite_pidfile',
>>> >> '_pidfile', '_should_restart', '_waitpid', 'daemonize',
>>> >> 'daemonize_complete', 'daemonize_start', 'die_if_already_running',
>>> >> 'errno', 'fcntl', 'get_detach', 'get_pidfile',
>>> >> 'ignore_existing_pidfile', 'is_chdir_enabled', 'logging',
>>> >> 'make_pidfile_name', 'os', 'ovs', 'parse_opt', 'read_pidfile',
>>> >> 'resource', 'set_detach', 'set_monitor', 'set_no_chdir',
>>> >> 'set_pidfile', 'signal', 'sys', 'time', 'usage']
>>> >> >>>
>>> >
>>> > I don't know whether that's informational or pointing out a bug.  If the
>>> > former, thanks.  If the latter, please be more specific: I don't know
>>> > Python well enough to see it.
>>> >
>>>
>>> Informational only. Will send another patch shortly.
>>>
>>> --
>>> Sajjad Lateef
>>> slateef at nicira.com
>>>
>>> _______________________________________________
>>> dev mailing list
>>> dev at openvswitch.org
>>> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
>>
>>
>
>
>
> --
> Sajjad Lateef
> slateef at nicira.com
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
>