Bug 505

Summary:	live555: new package
Product:	buildroot	Reporter:	Kelvin Cheung <keguang.zhang>
Component:	Other	Assignee:	unassigned
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	buildroot
Priority:	P3
Version:	unspecified
Target Milestone:	2010.05
Hardware:	All
OS:	Linux
URL:	http://www.live555.com/liveMedia/
Host:		Target:
Build:
Attachments:	live555: new package live555: new package (updated)

Description Kelvin Cheung 2009-07-30 06:39:35 UTC

LIVE555 Streaming Media forms a set of C++ libraries for multimedia
streaming, using open standard protocols (RTP/RTCP, RTSP, SIP).

See: http://www.live555.com/liveMedia/

Comment 1 Kelvin Cheung 2009-07-30 06:42:52 UTC

Created attachment 517 [details]
live555: new package

Comment 2 Will Newton 2009-07-30 09:53:24 UTC

This rule in the makefile looks wrong to me:

$(STAGING_DIR)/usr/include/live: $(LIVE555_DIR)/liveMedia/libliveMedia.a
        cp -a $(LIVE555_DIR) $(STAGING_DIR)/usr/include/

The whole source tree is copied into /usr/include in the staging_dir?

Comment 3 Thomas Petazzoni 2009-07-30 12:12:51 UTC

After a quick look at the patch :
 * I think we could make the config.arm-linux thing a little more generic, so as to support other architectures as well
 * The config.arm-linux should probably be generated, or at least sed'ed by the .mk file, to really use $(TARGET_CC), $(TARGET_CFLAGS), $(TARGET_LD), $(TARGET_LDFLAGS), etc. This would for example fix this package with external toolchains

Comment 4 Kelvin Cheung 2009-07-31 01:40:52 UTC

(In reply to comment #2)
> This rule in the makefile looks wrong to me:
> 
> $(STAGING_DIR)/usr/include/live: $(LIVE555_DIR)/liveMedia/libliveMedia.a
>         cp -a $(LIVE555_DIR) $(STAGING_DIR)/usr/include/
> 
> The whole source tree is copied into /usr/include in the staging_dir?
> 

Yes, the installation is a little strange.
The live555 does not provide 'make install', so we have to install it manually.

Copying the source tree into /usr/include is mainly for MPlayer-1.0rc2.

It will autodetect whether live555 (both header file and archive) was installed
to default or specified path.

See [Bug 507]: https://bugs.busybox.net/show_bug.cgi?id=507

Comment 5 Kelvin Cheung 2009-07-31 02:10:49 UTC

(In reply to comment #3)
> After a quick look at the patch :
>  * I think we could make the config.arm-linux thing a little more generic, so
> as to support other architectures as well
>  * The config.arm-linux should probably be generated, or at least sed'ed by the
> .mk file, to really use $(TARGET_CC), $(TARGET_CFLAGS), $(TARGET_LD),
> $(TARGET_LDFLAGS), etc. This would for example fix this package with external
> toolchains
> 

Now, the patch could support both x86 and arm.

Other architectures such as alpha and blackfin are already supported by
'genMakefiles' (something like configure).
But I do not have toolchains except x86 and arm.

Comment 6 Kelvin Cheung 2009-07-31 04:20:16 UTC

Created attachment 533 [details]
live555: new package (updated)

patch updated

Comment 7 Thomas Petazzoni 2010-02-21 22:33:00 UTC

Move milestone to 2010.05 since enhancements will not be handled before the release of 2010.02.

Comment 8 Maxime Ripard 2011-09-17 20:12:44 UTC

Commit 86885771d added this package