[pacman-dev] [PATCH 1/5] pacman-key: keyring management tool

[Allan McRae]

On 16/09/10 23:36, Allan McRae wrote:
> On 16/09/10 13:29, Denis A. Altoé Falqueto wrote:
>> On Wed, Aug 4, 2010 at 10:17 AM, Allan McRae<allan at archlinux.org> wrote:
>>> On 28/07/10 13:50, Denis A. Altoé Falqueto wrote:
>>>>
>>>> The script pacman-key will manage pacman's keyring. It imports,
>>>> exports,
>>>> fetches from keyservers, helps in the process of trusting and updates
>>>> the trust database.
>>>>
>>>> Signed-off-by: Denis A. Altoé Falqueto<denisfalqueto at gmail.com>
>>>
>>> Hi Denis,
>>>
>>> I think it would be good for us to focus on getting this onto the gpg
>>> branch
>>> and then move onto the other patches. I do not think this requires
>>> massive
>>> changes to be ready.
>>
>> Hi.
>>
>> Sorry for the delay again. Time is so short lately... It took way
>> longer than I would like. But here I am again. I'll answer only the
>> things that I would like to discuss further. The other points were
>> implemented as advised by you.
>>
>>>> +prepare_homedir() {
>>>> + if [[ ! -d "${PACMAN_KEYRING_DIR}" ]] ; then
>>>> + mkdir -p "${PACMAN_KEYRING_DIR}"
>>>> + touch "${PACMAN_KEYRING_DIR}/secring.gpg"
>>>> + touch "${PACMAN_KEYRING_DIR}/pubring.gpg"
>>>> + chmod 700 "${PACMAN_KEYRING_DIR}"
>>>> + chmod 600 "${PACMAN_KEYRING_DIR}"/{sec,pub}ring.gpg
>>>
>>> We should just use:
>>> install -dm700 ${PACMAN_KEYRING_DIR}
>>> to create the directory with the right permissions.
>>>
>>> And should those files actually be part of the pacman package and so
>>> guaranteed to be present.
>>
>> Yes, I believe the best place is pacman package. I removed the
>> function, so we need to make sure the PKGBUILD for pacman creates the
>> proper files and directory.
>>
>
> I will adjust the Makefile to ensure these directories get made.

It has become abundantly clear to me that my autotools knowledge is 
lacking and I have no idea how to actually do this...  especially the 
restricted permissions part.

Anyone else care to take a stab?

Allan