cvs commit: ports/mail/libvmime Makefile
Xin LI
delphij at delphij.net
Wed Jul 23 10:22:18 UTC 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Pav Lucistnik wrote:
| Xin LI píše v út 22. 07. 2008 v 19:19 -0700:
|> -----BEGIN PGP SIGNED MESSAGE-----
|> Hash: SHA1
|>
|> Pav Lucistnik wrote:
|> | pav 2008-07-23 00:57:45 UTC
|> |
|> | FreeBSD ports repository
|> |
|> | Modified files:
|> | mail/libvmime Makefile
|> | Log:
|> | - Mark BROKEN: does not compile with gnutls-2.4.x
|>
|> Er, why, how can we say this version of gnutls is not broken when they
|> have this?
|>
|> ~ typedef enum gnutls_certificate_print_formats
|> ~ {
|> ~ GNUTLS_CRT_PRINT_FULL,
|> ~ GNUTLS_CRT_PRINT_ONELINE,
|> ~ GNUTLS_CRT_PRINT_UNSIGNED_FULL, // extra ","
|> ~ } gnutls_certificate_print_formats_t;
|
| I don't mind if it's fixed in libvmime or in gnutls port, as long as it
| get fixed.
Unfortunately this can *NOT* be fixed in libvmime because gnutls's
include file is broken. By making an instance of
gnutls_certificate_print_formats_t the compiler would notice the problem
immediately.
| BTW it's still this way in gnutls git.
I have submitted ticket #770 on gnutls trac. Just because it is still
in gnutls git does not mean that it is correct. Can we fix the (broken)
security/gnutls, rather than mark (the non-broken) libvmime broken? The
port works just fine with patched gnutls.
Cheers,
- --
Xin LI <delphij at delphij.net> http://www.delphij.net/
FreeBSD - The Power to Serve!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (FreeBSD)
iEYEARECAAYFAkiHBisACgkQi+vbBBjt66BeTgCeMuPm7Nl9h7zP7wqiAiup0n9O
LL4AoMBR/B+nCtjd4meHnE8OklIuKAip
=A34Q
-----END PGP SIGNATURE-----
-------------- next part --------------
diff --git a/includes/gnutls/gnutls.h.in b/includes/gnutls/gnutls.h.in
index 13d507b..2025cdb 100644
--- a/includes/gnutls/gnutls.h.in
+++ b/includes/gnutls/gnutls.h.in
@@ -299,7 +299,7 @@ extern "C"
{
GNUTLS_CRT_PRINT_FULL,
GNUTLS_CRT_PRINT_ONELINE,
- GNUTLS_CRT_PRINT_UNSIGNED_FULL,
+ GNUTLS_CRT_PRINT_UNSIGNED_FULL
} gnutls_certificate_print_formats_t;
typedef enum
More information about the cvs-all
mailing list