Bug 906519 - /usr/include/sasl/sasl.h error: typedef 'sasl_malloc_t' is initialized (use decltype instead)
Summary: /usr/include/sasl/sasl.h error: typedef 'sasl_malloc_t' is initialized (use d...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: cyrus-sasl
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Petr Lautrbach
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-01-31 18:55 UTC by Rex Dieter
Modified: 2013-02-01 12:12 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-01-31 19:40:46 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
add #include <sys/types.h> for size_t (467 bytes, patch)
2013-01-31 19:22 UTC, Rex Dieter
no flags Details | Diff

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.


Note You need to log in before you can comment on or make changes to this bug.