Please upgrade ebox-dhcp to 0.12.2

Bug #321940 reported by Javier Uruen Val
4
Affects Status Importance Assigned to Milestone
ebox-dhcp (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

A new upstream version that contains bug fixes and improvements has been
released.

It can be downloaded from here:

http://www.ebox-platform.com/releases/sources/ebox-dhcp_0.12.2.orig.tar.gz

Changelog
0.12.2
    + Bugfix: two wrong calls to EBox::Model::Row API fixed. This
      prevented from adding fixed ranges after adding a fixed address
0.12.1
    + Bug fix: use the new EBox::Model::Row API for DHCP options
0.12
    + Make default and max leasing time mandatory fields
    + Add field help to model
    + Make custom_ns field required
    + Bugfix: Mark managers as changed every time they ask to change
    from/to static method because some models may appear/disappear
    because of modifying configuration method in interfaces
0.11.102
    + Bugfix. Logging were not working with hostnames with characters
    different from alphanumeric plus '_'
0.11.101
    + Bugfix. Secondary nameserver in options does not fill a default
    value when setting as undef by user
    + Added 'setLeases' method to set lease times from module API
    + Bugfix: Virtual interface options do not override real interface
    options anymore
0.11.100
    + onInstall() functionality moved to migration script
    + Do not remove dhcpd3 rc links
    + Stop dhcpd3 in pre-start in upstart script

Related branches

Revision history for this message
Javier Uruen Val (juruen) wrote :

Binaries can also be found in:

https://launchpad.net/~ebox-unstable/+archive

David Futcher (bobbo)
Changed in ebox-dhcp:
assignee: nobody → bobbo
importance: Undecided → Wishlist
status: New → In Progress
Changed in ebox-dhcp:
assignee: bobbo → nobody
status: In Progress → Confirmed
Revision history for this message
David Futcher (bobbo) wrote :

Javier: I have prepared an upload to update this. Do you want to wait and get sponsorship for your upload, or would you mind if I just upload mine?

Revision history for this message
Javier Uruen Val (juruen) wrote :

Upload it please :)

Revision history for this message
David Futcher (bobbo) wrote :

Your upload does more, so I'll look at sponsoring it :)

Revision history for this message
David Futcher (bobbo) wrote :

Javier. I have a few comments about the diff.gz that I feel need fixed before I can upload the package for you. Nothing particularly important, but just things it would be good to look at.

1 - You mention you drop the dpatch dependency, but don't give a rationale for removing it. I notice the debian/patches dir has been removed. Is the old patch no longer necessary? If so it would be better to have a changelog entry like:

  * Remove dpatch patchsys, as patch is no longer necessary
     - debian/control: Remove dpatch from Build-Depends
     - debian/patches: Remove all patches

2 - There is a lot of PPA entries in the ChangeLog that shouldnt be there. Also the changelog doesnt seem to match up with the current version in Ubuntu. Please make sure you are applying your changes against the latest version in Ubuntu (0.11.99-0ubuntu2).

3 - You mis-spelled "standards" in the changelog, minor, but better if it is fixed. There is also no need to have the word "Closes" before the launchpad bug number. That top line should read simply "* New upstream release. (LP: #321940)"

4 - You should put a Homepage field in debian/control.

Apart from that, the patch looks ok. Please re-subscribe ubuntu-universe-sponsors when you have dealt with the issues above.

Revision history for this message
Javier Uruen Val (juruen) wrote :

@David,

Thanks for your comments. I'm attaching a new diff.gz that fixes all the issues you brought up.

Revision history for this message
Iain Lane (laney) wrote :

Uploading, thanks for your contribution.

I fixed the watch file and removed the extra changelog entries from your diff.

Changed in ebox-dhcp:
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ebox-dhcp - 0.12.2-0ubuntu1

---------------
ebox-dhcp (0.12.2-0ubuntu1) jaunty; urgency=low

  [ Javier Uruen Val ]
  * New upstream release. (LP: #321940)
  * debian/control:
    - Bump standards version.
    - Remove dpatch from Build-Depends
    - Add homepage
  * debian/patches
    - Remove all patches as they are already included in upstream
  * debian/copyright
    - Update upstream URL
  * debian/watch:
    - Add watch file.
  * debian/ebox-dhcp.postinst
    - Fix indentation
    - Do not restart apache, ebox trigger takes care of it
  * debian/ebox-dhcp.postrm
    - Purge ebox-dhcp configuration

  [ Iain Lane ]
  * debian/watch: Fix to work with current upstream tarballs
  * debian/changelog: Remove entries that never entered Ubuntu

 -- Javier Uruen Val <email address hidden> Sun, 18 Jan 2009 18:19:19 +0100

Changed in ebox-dhcp:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.