LEVAI Daniel <l...@ecentrum.hu> writes: > On szo, dec 14, 2013 at 22:41:21 +0000, Stuart Henderson wrote: >> On 2013/12/14 18:35, LEVAI Daniel wrote: > [...] >> >> | MASTER_SITES = >> http://github.com/technion/${PROJECT}/archive/v${V}.tar.gz?dummy=/ >> >> Rather than this url query hack, please use the newer style of github >> handling you'll see in other ports e.g. www/tt-rss. > > Oh blimey! I would have sweared that I saw this in devel/libgit2 (that > pkg was even your example in [1] :) ).
I think that included the GH_VER stuff. >> >> | SHARED_LIBS += scrypt 1.0 # 1.0 >> >> Library versioning for OpenBSD ports should start at 0.0. > > Right. but you can keep the upstream version in the comment. >> | @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. This still needs SHARED_ONLY, the .a isn't installed and building it should not involve -fPIC. PFRAG.shared can be merged in PLIST. > > [1] http://marc.info/?l=openbsd-ports&m=138588950307176&w=2 -- jca | PGP: 0x06A11494 / 61DB D9A0 00A4 67CF 2A90 8961 6191 8FBF 06A1 1494