Skip to contentSkip to conversation

Series short description

PATCH In review Ian Kent 2 participants 7 messages Jul 24, 2013 Kernel automounter

lore
Share & tools
lore

Ian Kent

author maintainer
#0 lore raw

cover Series short description

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Signature · 1 line
Signature
#1 lore raw
Quoted 33 lines · Ian Kent

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Quoted 2 lines

--
Signature

Hi Ian:
Thanks.
But I haven't seen the commits published, am I use the wrong repo url? [1]

Dennis Lan (dlan)

[1] git://git.kernel.org/pub/scm/linux/storage/autofs/autofs.git

author maintainer
#2 lore raw
Quoted 34 lines · Dennis Lan (dlan)
Quoted 33 lines · Ian Kent

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Quoted 6 lines
Quoted 2 lines

--
Signature

Hi Ian:
Thanks.
But I haven't seen the commits published, am I use the wrong repo url? [1]

This was a false start, as I replied to it later.

The posts to follow are what I have in the queue for the next time I commit changes.

There are three types of patches in the queue of patches I have.

First are ones I will commit next time I do a commit, second are patches that may on may not be applied depending on their state and which may or may not be committed next commit, and third patches which I'm carrying that I don't intend to commit, at least for the moment.

So I'd like to confirm I have what your hoping for before committing anything and then there's the as yet unfinished patch ....

Ian

Quoted 5 lines

Dennis Lan (dlan)

[1] git://git.kernel.org/pub/scm/linux/storage/autofs/autofs.git

#3 lore raw
Quoted 54 lines · Ian Kent
Quoted 34 lines · Dennis Lan (dlan)
Quoted 33 lines · Ian Kent

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Quoted 6 lines
Quoted 2 lines

--
Signature

Hi Ian:
Thanks.
But I haven't seen the commits published, am I use the wrong repo url? [1]

This was a false start, as I replied to it later.

The posts to follow are what I have in the queue for the next time I commit changes.

There are three types of patches in the queue of patches I have.

First are ones I will commit next time I do a commit, second are patches that may on may not be applied depending on their state and which may or may not be committed next commit, and third patches which I'm carrying that I don't intend to commit, at least for the moment.

Quoted 12 lines

So I'd like to confirm I have what your hoping for before committing anything and then there's the as yet unfinished patch ....

Ian

Quoted 5 lines

Dennis Lan (dlan)

[1] git://git.kernel.org/pub/scm/linux/storage/autofs/autofs.git

HI Ian:
As I go through the patches, I think you've already include most of
my patches,
but expect this one:
[PATCH 1/6] fix compile error with heimdal support enabled So far so good, and I will have a test tomorrow, and will let you
know the result.

Thanks

Dennis Lan (dlan)

author maintainer
#4 lore raw
Quoted 68 lines · Dennis Lan (dlan)
Quoted 54 lines · Ian Kent
Quoted 34 lines · Dennis Lan (dlan)
Quoted 33 lines · Ian Kent

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Quoted 6 lines
Quoted 2 lines

--
Signature

Hi Ian:
Thanks.
But I haven't seen the commits published, am I use the wrong repo url? [1]

This was a false start, as I replied to it later.

The posts to follow are what I have in the queue for the next time I commit changes.

There are three types of patches in the queue of patches I have.

First are ones I will commit next time I do a commit, second are patches that may on may not be applied depending on their state and which may or may not be committed next commit, and third patches which I'm carrying that I don't intend to commit, at least for the moment.

Quoted 12 lines

So I'd like to confirm I have what your hoping for before committing anything and then there's the as yet unfinished patch ....

Ian

Quoted 5 lines

Dennis Lan (dlan)

[1] git://git.kernel.org/pub/scm/linux/storage/autofs/autofs.git

Quoted 7 lines

HI Ian:
As I go through the patches, I think you've already include most of
my patches,
but expect this one:
[PATCH 1/6] fix compile error with heimdal support enabled So far so good, and I will have a test tomorrow, and will let you
know the result.

Oh right, I did miss that one.

I'll fix that but maybe I should wait for a patch with the added error handling we discussed or would you just like me to fix it and add it to our list?

Quoted 4 lines

Thanks

Dennis Lan (dlan)

#5 lore raw
Quoted 77 lines · Ian Kent
Quoted 68 lines · Dennis Lan (dlan)
Quoted 54 lines · Ian Kent
Quoted 34 lines · Dennis Lan (dlan)
Quoted 33 lines · Ian Kent

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)
Quoted 6 lines
Quoted 2 lines

--
Signature

Hi Ian:
Thanks.
But I haven't seen the commits published, am I use the wrong repo url? [1]

This was a false start, as I replied to it later.

The posts to follow are what I have in the queue for the next time I commit changes.

There are three types of patches in the queue of patches I have.

First are ones I will commit next time I do a commit, second are patches that may on may not be applied depending on their state and which may or may not be committed next commit, and third patches which I'm carrying that I don't intend to commit, at least for the moment.

Quoted 12 lines

So I'd like to confirm I have what your hoping for before committing anything and then there's the as yet unfinished patch ....

Ian

Quoted 5 lines

Dennis Lan (dlan)

[1] git://git.kernel.org/pub/scm/linux/storage/autofs/autofs.git

Quoted 7 lines

HI Ian:
As I go through the patches, I think you've already include most of
my patches,
but expect this one:
[PATCH 1/6] fix compile error with heimdal support enabled So far so good, and I will have a test tomorrow, and will let you
know the result.

Quoted 6 lines

Oh right, I did miss that one.

I'll fix that but maybe I should wait for a patch with the added error handling we discussed or would you just like me to fix it and add it to our list?

Yes, Please help me to fix it. It's trivial. and Thanks!

Quoted 6 lines
Quoted 4 lines

Thanks

Dennis Lan (dlan)

author maintainer
#6 lore raw
In reply to Ian Kent

On Wed, 2013-07-24 at 16:17 +0800, Ian Kent wrote:

False start, ignore it.

Quoted 33 lines

The following series implements...

---

Dustin Polke (1):
autofs-5.0.7 - fix compilation of lookup_ldap.c without sasl

Ian Kent (1):
autofs-5.0.7 - use LIBS for link libraries

Lan Yixun (dlan) (7):
autofs-5.0.7 - fix typo forced-shutdown should be force-shutdown autofs-5.0.7 - fix hesiod check error and use correct $(LIBS) setting autofs-5.0.7 - fix dead LDAP symbolic link when LDAP support is disabled autofs-5.0.7 add missing libtirpc lib to mount_nfs.so when TIRPC enabled autofs-5.0.7 - use compiler determined by configure instead of hard-coded ones autofs-5.0.7 - remove hard-coded STRIP variable
autofs-5.0.7 - unbundle NOTSTRIP from DEBUG so they wont depend on each other

 CHANGELOG             |    4 ++++
 Makefile.conf.in      |    3 +++
 Makefile.rules        |   30 +++++++++++++++---------------
 aclocal.m4            |   12 ++++++------
 configure             |   20 ++++++++++----------
 configure.in          |    2 +-
 daemon/Makefile       |    2 +-
 include/lookup_ldap.h |    4 ++--
 lib/Makefile          |    4 ++++
 modules/Makefile      |   20 ++++++++++++--------
 modules/lookup_ldap.c |   14 ++++++++++++--
 11 files changed, 70 insertions(+), 45 deletions(-)