LEVAI Daniel <[email protected]> writes: > On cs, dec 19, 2013 at 14:14:40 +0100, Jérémie Courrèges-Anglas wrote: >> LEVAI Daniel <[email protected]> writes: > [...] >> but you can keep the upstream version in the comment. > [...] > > This was fixed in the last tarball.
"0" was indeed their version number, hehe. Ahem.
>> >> | @bin lib/libscrypt.so.0
>> >> | lib/libscrypt.so
>> >>
>> >> Library generation is wrong, there should be
>> >> libscrypt.so.<version_from_shared_libs_line> and also either a
>> >> static library, or make the port SHARED_ONLY.. I haven't built but
>> >> I guess one of these files is a symlink and one is an incorrectly
>> >> built library.
>> >
>> > Okay, I tried to do a better job this time. The source builds a
>> > libscrypt.so.0 file, and I couldn't get pkg_create(1) to recognize the
>> > lib mentioned in SHARED_LIBS, so I had to mv libscrypt.so.0 ->
>> > libscrypt.so.0.0
>> > I've also @comment-ed the symlink to ...so.0 in PLIST. Now the library
>> > is correctly prefixed with @lib in the PLIST.
>>
>> Just patch the Makefile. We don't use -soname. What I'd do is ''-o
>> libscrypt.so.${LIBscrypt_VERSION}'', you can pass the version in
>> MAKE_FLAGS. The symlinks can be patched away while here.
>
> Okay, thanks! I've attached a new archive that includes these changes.
I doubt -soname $fullversion is a good idea. But we don't need
it -soname at all. Also, duplicating information that could be present
in the port Makefile only isn't a good idea.
Here's a tarball that uses MAKE_FLAGS to pass CC, CFLAGS and
LIBscrypt_VERSION.
Also:
- simplify V / GH_VER usage (only GH_VER is needed)
- reorder variables to match Makefile.template
- provide a HOMEPAGE
- more info in DESCR
libscrypt.tgz
Description: Binary data
>> This still needs SHARED_ONLY, the .a isn't installed and building it >> should not involve -fPIC. >> >> PFRAG.shared can be merged in PLIST. > > These were also fixed in the last tarball. Boo, tarball mixup. > Daniel -- jca | PGP: 0x06A11494 / 61DB D9A0 00A4 67CF 2A90 8961 6191 8FBF 06A1 1494
