>>>>> "stefan" == Stefan van der Eijk <[EMAIL PROTECTED]> writes:

>>> Otherwise we'll have a dead symlink in /usr/lib, which points to
>>> the .so symlink in /lib.
>>> 
>> 
>> Then fix the symlink in %{_libdir}. Don't populate /%{_lib} with .so
>> symlinks.
>> 
stefan> Fine with me, as long as xfsdump will build.

>> And I insist on %{_lib} & %{_libdir}. I had fixed that for lib64 but
>> this magically vanished.
>> 
stefan> Yes, I agree. Sorry about that.

>>> Any objections to updating xfsdump to 2.2.3?
>>> 
>> 
>> I don't know, kernel people will answer. And I suppose Vincent
>> already issued an update to 2.2.3.
>> 
>> 
>>>> BuildRequires:        libattr-devel
>>>> BuildRequires:        libdm-devel
>>>> BuildRequires:        libext2fs-devel
>>>> BuildRequires:        libxfs-devel
>>>> 
>> 
>> Always BuildRequires: <thing>-devel forms only, not lib<thing>-devel.
>> 
stefan> Sure, but:

ok, will fix in teh same batch.

/me decides to spend the rest of the evening recompiling all his
packages.

/me thinks that this is optimist.

stefan> $ urpmq attr-devel
stefan> no package named attr-devel

stefan> $ urpmq dm-devel
stefan> no package named dm-devel

stefan> $ urpmq ext2fs-devel
stefan> no package named ext2fs-devel

stefan> $ urpmq xfs-devel
stefan> no package named xfs-devel

stefan> I understand where this is coming from, and I agree that it needs to
stefan> be done. But I don't think that it needs to be brought up in this
stefan> discussion. I'm getting the feeling that somebody is looking for
stefan> something to pick on...

stefan> I suggest:

stefan> *  rpmlint is adjusted to warn about (& enforce?) this on uploads;
stefan> * a document is written describing these changes so this can be
stefan> finalized into some sort of  guideline / policy. So people can
stefan> read it through, understand how it works and how it should be
stefan> implemented in the packages;


There is a document:

http://people.mandrakesoft.com/~gc/<somewhere>

stefan> For instance: I don't understand how the mklib stuff works and how I
stefan> should fix the rpmlint errors I get on the topic.

Just fixed it for dmapi, it is trivial:

%define lib_name_orig %mklibname dm

instead of

%define lib_name_orig libdm

Rule of thumb:
- when you have a problem of that kind, look for a package maintained
  by gb about how it is done there :p

Later, Juan.

-- 
In theory, practice and theory are the same, but in practice they 
are different -- Larry McVoy

Reply via email to