[Pkg-shadow-devel] cdbs and questions

[Christine Spang]

OK - I've finally gotten around to (almost) finishing the port to cdbs.
(As to Martin's concerns from earlier: I don't mind helping in whatever
way I can if issues crop up with cdbs, though this may not be much. In
what way is it 'not really maintained'? It seems to have an Alioth
maintainer group, and its last upload was only a little over a month
ago, though the upload only added documentation.)

I've committed a new patch that makes all the necessary changes in order
to build using cdbs, so it's easy to use it or lose it with no harm
done. Please break it, comment, suggest. (Though I seem to already have
done #1.)

First: When building just the clean 4.0.12 package, both with and
without cdbs, I get the following error when using 'su':

christine at Freyja:/var/cache/pbuilder/result$ su
Password:
configuration error - unknown item 'FTMP_FILE' (notify administrator)
configuration error - unknown item 'CLOSE_SESSIONS' (notify
administrator)
configuration error - unknown item 'CLOSE_SESSIONS' (notify
administrator)
root at Freyja:/var/cache/pbuilder/result#

Though, the cdbs version doesn't allow the user change after the errors
(probably a mistake somewhere on my part):

christine at Freyja:/var/cache/pbuilder/result$ su
Password:
su: Authentication failure
configuration error - unknown item 'FTMP_FILE' (notify administrator)
configuration error - unknown item 'CLOSE_SESSIONS' (notify
administrator)
Sorry.
christine at Freyja:/var/cache/pbuilder/result$

---------------------------------------------------------------------

Other stuff -

Patch Notes:

dh_movefiles -> dh_install

cdbs uses dh_install rather than dh_movefiles for file installation.
Switched.

debian/control

Added cdbs to Build-Depends. Doing so through a patch is all sorts of
Doesn't Work (TM), so I just added it manually.

Building with Sarge:

We used to have a chunk of makefile added to the end of debian/rules to
make up for the lack of quilt.mk in the Sarge quilt package. However,
there -is- the quilt cdbs makefile fragment in the Sarge package, so we
should build alright even on Sarge. I haven't tested it, though.

Extras in /usr/share/doc:

The new packages contain NEWS.gz, README, TODO, and changelog.gz in both
login and passwd. Is there any reason that they're -not- in the previous
packages? If so, they can easily be excluded. (Though it's a little
redundant right now - the exact same files are going into both packages.
Though that's the same as already happens with the Debian changelog.)

dh_installpam

We could use this to clean up the installation of PAM files. However, I
don't see a very clean way to use it as it now stands, and especially
without moving all the pam files out of debian/pam.d into plain debian/.

dh_link

I haven't done this yet, but we could probably get rid of the manual
symlinking, with {package}.links files.

Weird stuff:

Right now the build with is running ./configure -twice- for reasons
unknown, once after running autoconf and a later 'recheck'. I suspect
that it has something to do with cdbs + updating autoconf, etc. Will
look into this - not sure what it's for.

Also, all the other manual installing of files in debian/ is just
screaming 'automate me!' or 'clean me!', though this may just be me.
Some of the above could help.

-----------------------------------------------------------------

Questions regarding the old debian/rules:

-Why is DH_COMPAT exported twice? Does this mean that the first part of
debian/rules depends on debhelper >= 4, whereas the rest can use
debhelper >= 3? (this doesn't seem to make sense) Was the second one
just left in accidentally, as the comment on the first one seems to
suggest?

-What is this for?:

ifneq ($(DEB_HOST_ARCH_OS),hurd)
	echo "loginpam=login (>= 970502-1), libpam-modules (>= 0.72-5)" >>
debian/passwd.substvars
else
	echo "loginpam=login, libpam-modules (>= 0.72-5)" >>
debian/passwd.substvars
endif

Did we ever figure out the cause of all the warnings that lintian and
linda are giving out? (Sorry if I wasn't paying attention and missed
it.)

Anyway, even if we don't end up using this, I learned a lot doing the
work, gee. :)

-- 
A. Christine Spang
http://frisia.middle-earth.us
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.alioth.debian.org/pipermail/pkg-shadow-devel/attachments/20050821/4a16fb52/attachment.pgp