Hi,

On Thu, 1 Mar 2007 [EMAIL PROTECTED] wrote:

> I'm resending this patch because it hasn't been committed nor commented 
> so far and the problem it fixes is a blocker for distributors.

Sorry I wasn't aware this is important for you (you can have e.g. a patch 
section in rpm .spec files which is exactly designed partly to solve such 
source -> binary packaging issues) and unfortunatley I didn't and don't 
have the time right now to explain why both of your patches are basically 
incorrect. It works for __your__ but it breaks working scenarios for 
__others__.
 
> Binary packages have to run ldconfig in their postinstall scripts.

Bingo! When you silence, ignore the error automatically then you hide an 
important step which may be forgotten by packagers. We do receive such 
errors after binary package installations.

> For the records: There are alternative ways to fix this. I've checked the 
> binary packages from Fedora, Mandriva, Debian and Ubuntu and they are all 
> simply removing the $(LDCONFIG) line, obviously breaking the purpose of 
> it. This patch is a solution that should work for both use cases: DESTDIR 
> packaging and "normal" builds. 

It doesn't always work. Some people want it even with DESTDIR, or don't 
want it during 'make install'. So it should be either a configure option to 
run or not to run during make install, or it should be solved in the binary 
package alongside running ldconfig during postinstallation.

> I'd like to see this fixed upstream instead of distribution-specific 
> local hacks.

I don't think it's a distribution-specific local hack but a step for binary 
packaging. The configure option (default to run ldconfig) would be also 
welcome and it would be applied right away.

        Szaka

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
ntfs-3g-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ntfs-3g-devel

Reply via email to