Bug 906519

Summary: /usr/include/sasl/sasl.h error: typedef 'sasl_malloc_t' is initialized (use decltype instead)
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: cyrus-saslAssignee: Petr Lautrbach <plautrba>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: plautrba, tmraz, vanmeeuwen+fedora
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-31 19:40:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
add #include <sys/types.h> for size_t none

Description Rex Dieter 2013-01-31 18:55:03 UTC
kdepimlibs built ok prior to cyrus-sasl-2.1.26-1.fc19 landing in rawhide, now see:

In file included from /builddir/build/BUILD/kdepimlibs-4.10.0/kioslave/smtp/command.h:37:0,
                 from /builddir/build/BUILD/kdepimlibs-4.10.0/kioslave/smtp/command.cpp:32:
/usr/include/sasl/sasl.h:228:35: error: typedef 'sasl_malloc_t' is initialized (use decltype instead)
 typedef void *sasl_malloc_t(size_t);
                                   ^
/usr/include/sasl/sasl.h:228:29: error: 'size_t' was not declared in this scope
 typedef void *sasl_malloc_t(size_t);
                             ^
/usr/include/sasl/sasl.h:229:29: error: typedef 'sasl_calloc_t' is initialized (use decltype instead)
 typedef void *sasl_calloc_t(size_t, size_t);
                             ^
/usr/include/sasl/sasl.h:229:29: error: 'size_t' was not declared in this scope
/usr/include/sasl/sasl.h:229:37: error: 'size_t' was not declared in this scope
 typedef void *sasl_calloc_t(size_t, size_t);
                                     ^
/usr/include/sasl/sasl.h:230:38: error: 'size_t' has not been declared
 typedef void *sasl_realloc_t(void *, size_t);
                                      ^
/usr/include/sasl/sasl.h:233:33: error: variable or field 'sasl_set_alloc' declared void
 LIBSASL_API void sasl_set_alloc(sasl_malloc_t *,
                                 ^
/usr/include/sasl/sasl.h:233:33: error: 'sasl_malloc_t' was not declared in this scope
/usr/include/sasl/sasl.h:233:48: error: expected primary-expression before ',' token
 LIBSASL_API void sasl_set_alloc(sasl_malloc_t *,
                                                ^
/usr/include/sasl/sasl.h:234:5: error: 'sasl_calloc_t' was not declared in this scope
     sasl_calloc_t *,
     ^
/usr/include/sasl/sasl.h:234:20: error: expected primary-expression before ',' token
     sasl_calloc_t *,
                    ^
/usr/include/sasl/sasl.h:235:20: error: expected primary-expression before '*' token
     sasl_realloc_t *,
                    ^
/usr/include/sasl/sasl.h:235:21: error: expected primary-expression before ',' token
     sasl_realloc_t *,
                     ^
/usr/include/sasl/sasl.h:236:17: error: expected primary-expression before '*' token
     sasl_free_t *);
                 ^
/usr/include/sasl/sasl.h:236:18: error: expected primary-expression before ')' token
     sasl_free_t *);
                  ^

Comment 1 Rex Dieter 2013-01-31 19:03:36 UTC
Looks like the failure here is a variant of,
http://lists.andrew.cmu.edu/pipermail/cyrus-sasl/2012-December/002572.html

Comment 2 Rex Dieter 2013-01-31 19:06:33 UTC
I took the liberty of untagging cyrus-sasl-2.1.26-1.fc19 for now, until there's some resolution to the problem.

Comment 3 Rex Dieter 2013-01-31 19:22:41 UTC
Created attachment 691184 [details]
add #include <sys/types.h> for size_t

suggested minimal patch to fix issue of undefined size_t in sasl.h

Comment 4 Rex Dieter 2013-01-31 19:40:46 UTC
Fix committed,

%changelog
* Thu Jan 31 2013 Rex Dieter <rdieter> 2.1.26-2
- sasl.h: +#include<sys/types.h> for missing size_t type (#906519)
- tighten subpkg deps via %%?_isa


I apologize profusely and owe you an adult beverage (or 2) if you don't like this fix.

Comment 5 Petr Lautrbach 2013-02-01 12:12:25 UTC
I'm ok with it, thanks.