Bug 189775 - Review Request: dnsmasq - A lightweight DHCP/caching DNS server
Summary: Review Request: dnsmasq - A lightweight DHCP/caching DNS server
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-04-24 15:41 UTC by Jima
Modified: 2011-08-30 19:05 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-04-26 00:17:33 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Jima 2006-04-24 15:41:00 UTC
Spec URL: http://beer.tclug.org/fedora-extras/dnsmasq/dnsmasq.spec
SRPM URL: http://beer.tclug.org/fedora-extras/dnsmasq/dnsmasq-2.30-1.src.rpm
Description: 
Dnsmasq is lightweight, easy to configure DNS forwarder and DHCP server. It is designed to provide DNS and, optionally, DHCP, to a small network. It can serve the names of local machines which are not in the global DNS. The DHCP server integrates with the DNS server and allows machines with DHCP-allocated addresses to appear in the DNS with names configured either in each host or in a central configuration file. Dnsmasq supports static and dynamic DHCP leases and BOOTP for network booting of diskless machines.

Comment 1 Jochen Schmitt 2006-04-24 16:57:37 UTC
Good:
+ Local build works fine.
+ source tar has the same md5sum as the upstream.
+ Mock build works fine.


Bad:
- Requires should be:
Requires(post):  /sbin/chkconfig
Requires(post):  /sbin/service
Requires(preun): /sbin/chkconfig
Requires(preun): /sbin/service
- Debuginfo package contains no files.
- Error message when starting services:
Starting dnsmasq: dnsmasq: ISC dhcpd integration not available: set
HAVE_ISC_READER in src/config.h



Comment 2 Jima 2006-04-24 20:05:32 UTC
Thanks for reviewing.  Got some fixes for it.

Spec URL: http://beer.tclug.org/fedora-extras/dnsmasq/dnsmasq.spec
SRPM URL: http://beer.tclug.org/fedora-extras/dnsmasq/dnsmasq-2.30-2.src.rpm

Also uses
http://beer.tclug.org/fedora-extras/dnsmasq/dnsmasq-2.30-fedora-extras.patch .

All aforementioned issues should be resolved.  Let me know.

Comment 3 Jochen Schmitt 2006-04-25 15:55:52 UTC
God:
+ Build fine in mock.
+ Local install and remove work fine.
+ Starting and stoping of the service worked fine.
+ rpmlint of binary rpm ok.
+ rpmlint of source rpm ok.

APPROVED !!!

Comment 4 Jima 2006-04-26 00:17:33 UTC
Built on devel, FC5, FC4, and (after a bit of work) FC3.

Thanks!

Comment 5 Jima 2006-04-27 21:06:28 UTC
The issue in Comment #1 is the result of the default configuration triggering a
deprecated feature in dnsmasq.  The solution the error message suggests
(defining HAVE_ISC_READER) is a hack to re-enable that feature.  The dnsmasq
author asked me quite kindly to not enable the feature, as he'd rather not get
any more users hooked on a feature which will be going away eventually.

The workaround (which you should be doing anyway) is to set dhcp-range and
dhcp-leasefile in the configuration file.

I've released an update (2.30-3) which doesn't have the feature enabled, and
also cleans up my slightly hacked patch situation.  It should be pushed out to
mirrors shortly.

Comment 6 Jima 2006-05-01 23:29:25 UTC
*** Bug 190379 has been marked as a duplicate of this bug. ***

Comment 7 Douglas Schilling Landgraf 2011-08-30 18:39:16 UTC
Package Change Request
======================
Package Name: dnsmasq
New Branches: EL-6
Owners: dougsland

Comment 8 Gwyn Ciesla 2011-08-30 18:56:29 UTC
Git done (by process-git-requests).

Comment 9 Douglas Schilling Landgraf 2011-08-30 19:05:53 UTC
Hello Jon,

Thanks, you are so fast dude!


Note You need to log in before you can comment on or make changes to this bug.