Eli Zaretskii <eliz@gnu.org> writes:

>> > Each defcustom should have a :version tag stating the first version in
>> > which it will appear.
>>
>> Thanks, I have added the version now and removed the group, since I
>> remembered from another thread that there preference is not to declare
>> the group explicitly.
>>
>> P.S. While testing this I also fixed another issue where
>> `pakistan--regenerate-translations` will work the first time but not
>> subsequent ones. The fix was to replace
>>
>> (quail-install-map `(nil))
>>
>> (quail-install-map (list nil))
>>
>> and I don't understand why that makes a difference. But while trying to
>> debug this I noticed that `quail-install-map` ignores the optional NAME
>> argument and always installs the map in the current quail package. Is
>> this the intended behavior? I have now added a
>>
>> (quail-select-package "urdu-custom")
>>
>> before the call to `quail-install-map` to make sure regenerating the
>> input method in custom doesn't trample some other input method.
>
> Did you forget to attach the updated patch?

Yes, sorry about that. I have attached it now.

Rahguzar