Merge lp:~hingo/drizzle/drizzle-remove-Wall-in-autorunsh into lp:~drizzle-trunk/drizzle/development

[Henrik Ingo]

Recent autoconf changes by Brian made config/autorun.sh fail spectacularly on (K)Ubuntu 11.10. Removing -Wall makes it work again.

[Mark Atwood (fallenpegasus)]

It passed on http://jenkins.drizzle.org/view/Drizzle-build/job/drizzle-build-ubuntu11.10-64bit/ without this patch.

review: Needs Information

[Henrik Ingo (hingo)]

Hmm... good point. Do the buildslaves have standard autotools installed?

Note that the same warnings do happen:
configure.ac:143: the top level
configure.ac:143: warning: The macro `AC_TRY_LINK' is obsolete.
configure.ac:143: You should run autoupdate.

For some reason this results in a failure on my workstation, but apparently the build continues on the slave.

[Henrik Ingo (hingo)]

Ok, working from newest trunk works for me now too. I get the warnings but it doesn't error out.

What is the process for invalidating? Should I remove this branch?

review: Disapprove

[Stewart Smith (stewart)]

On Fri, 10 Feb 2012 15:01:24 -0000, Henrik Ingo <email address hidden> wrote:
> Review: Disapprove
>
> Ok, working from newest trunk works for me now too. I get the warnings but it doesn't error out.
>
> What is the process for invalidating? Should I remove this branch?

You can set the status of the merge req to 'rejected'

--
Stewart Smith

[Henrik Ingo (hingo)]

Actually, I can't. I have just Work In Progress, Needs Review and Merged. (It seems like Launchpad doesn't allow me to give up :-)

[Stewart Smith (stewart)]

On Sun, 12 Feb 2012 07:13:19 -0000, Henrik Ingo <email address hidden> wrote:
> Actually, I can't. I have just Work In Progress, Needs Review and
> Merged. (It seems like Launchpad doesn't allow me to give up :-)

hrrm... maybe this is an option that's reserved for the drizzle-trunk
group... so Brian or Mark or someone can do it :)

--
Stewart Smith

Unmerged revisions

2504. By Henrik Ingo on 2012-02-08

autoreconf -Wall creates a lot of trouble on (K)Ubuntu 11.10.
Removing -Wall helps.

=== modified file 'config/autorun.sh'
--- config/autorun.sh	2012-01-29 09:10:32 +0000
+++ config/autorun.sh	2012-02-08 22:44:20 +0000
@@ -25,4 +25,4 @@
 #  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
 #  SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-autoreconf --install --force --verbose -Wall
+autoreconf --install --force --verbose