Merge lp:~yolanda.robla/ubuntu/saucy/bind9/server_banner into lp:ubuntu/saucy/bind9
Proposed by
Yolanda Robla
Status: | Work in progress |
---|---|
Proposed branch: | lp:~yolanda.robla/ubuntu/saucy/bind9/server_banner |
Merge into: | lp:ubuntu/saucy/bind9 |
Diff against target: |
91 lines (+21/-3) 6 files modified
Makefile.in (+2/-0) bin/dig/Makefile.in (+3/-1) bin/dig/host.c (+1/-1) configure.in (+3/-0) debian/changelog (+10/-0) debian/control (+2/-1) |
To merge this branch: | bzr merge lp:~yolanda.robla/ubuntu/saucy/bind9/server_banner |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
LaMont Jones (community) | Disapprove | ||
Jamie Strandboge | Disapprove | ||
James Page | Needs Information | ||
Review via email: mp+171513@code.launchpad.net |
Description of the change
Updated to reflect distribution in server banner: https:/
To post a comment you must log in.
Unmerged revisions
- 59. By Yolanda Robla
-
updated way to get distribution
- 58. By Ubuntu <email address hidden>
-
* Updated to reflect distribution in banner:
- Makefile.in: include @BIND9_DISTRIBUTION@
- bin/dig/Makefile. in: include @BIND9_ DISTRIBUTION@
- bin/dig/host.c: used BIND9_DISTRIBUTION in descriptive text
- configure.in: set BIND9_DISTRIBUTION based on lsb_release
- debian/control: added lsb-release as build dependency
Hi Yolanda
Quick question - is this correct?
54 +if test -x /usr/bin/ lsb_release; then ION="BIND9_ DISTRIBUTION= $(lsb_release -si)"
55 + BIND9_DISTRIBUT
56 +else
The inclusion of BIND9_DISTRIBUTION= within the "" looked odd to me?