Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
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.