Merge lp:~ignacio-nin/percona-pam-for-mysql/percona-pam-for-mysql into lp:percona-pam-for-mysql

Proposed by Ignacio Nin
Status: Superseded
Proposed branch: lp:~ignacio-nin/percona-pam-for-mysql/percona-pam-for-mysql
Merge into: lp:percona-pam-for-mysql
Diff against target: 43 lines (+8/-14)
1 file modified
Makefile (+8/-14)
To merge this branch: bzr merge lp:~ignacio-nin/percona-pam-for-mysql/percona-pam-for-mysql
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Needs Fixing
Review via email: mp+79113@code.launchpad.net

This proposal has been superseded by a proposal from 2011-10-17.

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

(To repeat what was said on IRC)
OK with the hardcoded include path replaced by mysql_config --include

review: Needs Fixing
7. By Ignacio Nin

Use mysql_config --include

Get the include path for the mysql files from mysql_config instead of
having it hard-coded.

8. By Ignacio Nin

Migration to autotools

Migration to autotools for the project to take advantage of automake
automation, etc..

9. By Ignacio Nin

RPM integration

Include a spec file for creating RPMs.

10. By Ignacio Nin

Add a script for binary distribution

Add a script for creating binary distributions.
Include a check for pam in the configure script.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Makefile'
--- Makefile 2011-06-22 13:49:35 +0000
+++ Makefile 2011-10-12 12:34:23 +0000
@@ -1,21 +1,18 @@
1MASTER_SITE:=http://www.percona.com/downloads/community1CC:=gcc
2MYSQL_VERSION:=5.5.122CPPFLAGS:=-isystem /usr/include/mysql
3MYSQL_DIST:=mysql-$(MYSQL_VERSION).tar.gz3CFLAGS:=-fPIC -O3 -g -Wall -Wextra -Werror -fno-strict-aliasing
4
4PLUGINS:=auth_pam.so test_auth_pam_client.so5PLUGINS:=auth_pam.so test_auth_pam_client.so
56
6all : $(PLUGINS)7all : $(PLUGINS)
78
8$(MYSQL_DIST):
9 wget $(MYSQL_DIST);
10
11unpack: $(MYSQL_DIST)
12 -rm -rf mysql-$(MYSQL_VERSION)
13 tar zxf $(MYSQL_DIST)
14
15clean:9clean:
16 -rm $(PLUGINS) auth_pam.o lib_auth_pam_client.o test_auth_pam_client.o10 -rm $(PLUGINS) auth_pam.o lib_auth_pam_client.o test_auth_pam_client.o
1711
18.PHONY : unpack all clean12install: $(PLUGINS)
13 mysql_config --plugindir && install $(PLUGINS) $$(mysql_config --plugindir)
14
15.PHONY : unpack all clean install
1916
20%.so : %.o17%.so : %.o
21 ld -shared $^ -o $@ -lpam18 ld -shared $^ -o $@ -lpam
@@ -23,9 +20,6 @@
23auth_pam.so: auth_pam.o lib_auth_pam_client.o 20auth_pam.so: auth_pam.o lib_auth_pam_client.o
24test_auth_pam_client.so: test_auth_pam_client.o lib_auth_pam_client.o21test_auth_pam_client.so: test_auth_pam_client.o lib_auth_pam_client.o
2522
26CC:=gcc
27CPPFLAGS:=-isystem mysql-$(MYSQL_VERSION)/include
28CFLAGS:=-fPIC -O3 -g -Wall -Wextra -Werror
2923
30auth_pam.o: auth_pam.c lib_auth_pam_client.h24auth_pam.o: auth_pam.c lib_auth_pam_client.h
31lib_auth_pam_client.o: lib_auth_pam_client.c lib_auth_pam_client.h25lib_auth_pam_client.o: lib_auth_pam_client.c lib_auth_pam_client.h

Subscribers

People subscribed via source and target branches