Bug 189775

Summary:	Review Request: dnsmasq - A lightweight DHCP/caching DNS server
Product:	[Fedora] Fedora	Reporter:	Jima <jima>
Component:	Package Review	Assignee:	Jochen Schmitt <jochen>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	dougsland, igeorgex
Target Milestone:	---	Flags:	gwync: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-04-26 00:17:33 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	163779

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!