Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 454504

Summary: >=media-sound/mpg123-1.14.4 x32 ABI built with wrong LFS alias
Product: Gentoo Linux Reporter: camper <theonetruecamper>
Component: Current packagesAssignee: Gentoo Sound Team <sound>
Status: RESOLVED FIXED    
Severity: normal CC: thomas-forum
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 393673    
Attachments: fix configure/configure.ac to pick up lfs-alias from sizeof(off_t) instead of sizeof(long)

Description camper 2013-01-29 10:20:52 UTC
mpg123 is built with the wrong LFS-aliases, leading missing symbols when linking against other applications, e.g. mplayer2.

upstream bug: https://sourceforge.net/p/mpg123/feature-requests/37/

Reproducible: Always




partial configure output:

checking for special C compiler options needed for large files... no
checking for _FILE_OFFSET_BITS value needed for large files... no
checking for size_t... yes
checking for uintptr_t... yes
checking for ssize_t... yes
checking for off_t... yes
checking for int32_t... yes
checking for uint32_t... yes
checking for int16_t... yes
checking for uint16_t... yes
checking size of size_t... 4
checking size of ssize_t... 4
checking size of off_t... 8
checking size of int32_t... 4
checking size of long... 4
checking if we want to enable alias wrappers for largefile... yes
...
  mpg123 1.14.4

  Install path ............ /usr
  CPU Optimization ........ x86-64
  Compiler Optimization ... 0
  Gapless Support ......... enabled
  Debugging ............... disabled
  Seek table size ......... 1000
  FIFO support ............ enabled
  Buffer .................. enabled
  Network (http streams) .. enabled
  Network Sockets ......... Posix
  IPv6 (getaddrinfo) ...... enabled
  File offsets ............ default
  LFS alias symbols ....... enabled (32)
  Alignment checks ........
Comment 1 camper 2013-01-29 10:22:35 UTC
Created attachment 337202 [details, diff]
fix configure/configure.ac to pick up lfs-alias from sizeof(off_t) instead of sizeof(long)
Comment 2 SpanKY gentoo-dev 2013-03-03 03:51:22 UTC
Comment on attachment 337202 [details, diff]
fix configure/configure.ac to pick up lfs-alias from sizeof(off_t) instead of sizeof(long)

looks good to me.  have you sent this upstream ?  i don't see it here:
http://sourceforge.net/p/mpg123/bugs/
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2014-05-27 14:23:19 UTC
(In reply to SpanKY from comment #2)
> Comment on attachment 337202 [details, diff] [details, diff]
> fix configure/configure.ac to pick up lfs-alias from sizeof(off_t) instead
> of sizeof(long)
> 
> looks good to me.  have you sent this upstream ?  i don't see it here:
> http://sourceforge.net/p/mpg123/bugs/

Upstream tends to also look in out bugzilla, therefore CCing him here too.
Comment 4 Thomas Orgis 2014-05-27 17:56:37 UTC
Hey, I'm looking here when prodded ... my life is not _that_ empty that I browse the gentoo bugzilla on a daily basis.

I closed this issue in october of last year. Didn't year a complaint. The x32 ABI should work since mpg123-1.16.0 . We're at 1.20.0, currently.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2014-05-27 18:57:46 UTC
(In reply to Thomas Orgis from comment #4)
> Hey, I'm looking here when prodded ... my life is not _that_ empty that I
> browse the gentoo bugzilla on a daily basis.

sorry, I meant to say "look in our bugzilla", not "look in out bugzilla", and of course I meant only when prodded :)
Comment 6 Samuli Suominen (RETIRED) gentoo-dev 2014-05-27 18:59:23 UTC
(In reply to Thomas Orgis from comment #4)
> I closed this issue in october of last year. Didn't year a complaint. The
> x32 ABI should work since mpg123-1.16.0 . We're at 1.20.0, currently.

cool.

1.20.0 is in Portage, and current stable is 1.18.1something in Gentoo, so I believe nothing is left here