* [Buildroot] vlc: new package
@ 2013-03-14 19:46 Алексей Бродкин
2013-03-18 9:31 ` Thomas Petazzoni
0 siblings, 1 reply; 2+ messages in thread
From: Алексей Бродкин @ 2013-03-14 19:46 UTC (permalink / raw)
To: buildroot
Ismael,
I saw your patch for addition of vlc in Buildroot you posted back in
July 2012 here http://patchwork.ozlabs.org/patch/171335/
Since then there were no changes done so patch was not accepted.
I've done some minor changes for it to build successfully vlc in
buildroot from latest git.
You may find my changes in-lined below.
And at this point I'm wondering on how to proceed to get vlc support
in buildroot finally.
I see these options:
1. You may incorporate my changes in your patch and post v7 in mailing list.
2. I may do this re-spin and post v7 on your behalf.
3. Or I may start from scratch based on your patch, split changes on
small parts (as was suggested by Thomas earlier) and post them
one-by-one.
Please let me know if you're still interested in the subject and which
way is preferable to you.
Regards,
Alexey
My changes:
==================
diff --git a/package/multimedia/vlc/Config.in b/package/multimedia/vlc/Config.in
index cdeb751..bf81d5b 100644
--- a/package/multimedia/vlc/Config.in
+++ b/package/multimedia/vlc/Config.in
@@ -1,5 +1,8 @@
menuconfig BR2_PACKAGE_VLC
bool "vlc"
+ select BR2_PACKAGE_ZLIB
+ depends on BR2_INSTALL_LIBSTDCPP
+ depends on BR2_USE_WCHAR
help
VLC is a free and open source cross-platform multimedia player and
framework that plays most multimedia files as well as DVD, Audio CD,
@@ -111,6 +114,8 @@ config BR2_PACKAGE_VLC_TREMOR
config BR2_PACKAGE_VLC_AVCODEC
bool "FFmpeg (Many codecs)"
+ depends on BR2_LARGEFILE
+ depends on BR2_INET_IPV6
select BR2_PACKAGE_FFMPEG
help
There is a large number of codecs in FFmpeg/libavcodec.
diff --git a/package/multimedia/vlc/vlc.mk b/package/multimedia/vlc/vlc.mk
index 6a2a529..1caf709 100644
--- a/package/multimedia/vlc/vlc.mk
+++ b/package/multimedia/vlc/vlc.mk
@@ -24,6 +24,10 @@ VLC_CONF_OPT += --disable-rpath --enable-run-as-root \
--disable-dc1394 \
--disable-bluray
+VLC_CFLAGS = $(TARGET_CFLAGS)
+VLC_CFLAGS += -std=gnu99
+
+
ifeq ($(BR2_PACKAGE_VLC_ALSA),y)
VLC_DEPENDENCIES += alsa-lib
VLC_CONF_OPT += --enable-alsa
@@ -363,4 +367,4 @@ define VLC_FIX_LA_FILES
endef
VLC_POST_BUILD_HOOKS += VLC_FIX_LA_FILES
-$(eval $(call AUTOTARGETS))
+$(eval $(autotools-package))
==================
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Buildroot] vlc: new package
2013-03-14 19:46 [Buildroot] vlc: new package Алексей Бродкин
@ 2013-03-18 9:31 ` Thomas Petazzoni
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2013-03-18 9:31 UTC (permalink / raw)
To: buildroot
Dear ??????? ???????,
On Thu, 14 Mar 2013 23:46:26 +0400, ??????? ??????? wrote:
> I saw your patch for addition of vlc in Buildroot you posted back in
> July 2012 here http://patchwork.ozlabs.org/patch/171335/
> Since then there were no changes done so patch was not accepted.
>
> I've done some minor changes for it to build successfully vlc in
> buildroot from latest git.
> You may find my changes in-lined below.
>
> And at this point I'm wondering on how to proceed to get vlc support
> in buildroot finally.
> I see these options:
>
> 1. You may incorporate my changes in your patch and post v7 in
> mailing list. 2. I may do this re-spin and post v7 on your behalf.
> 3. Or I may start from scratch based on your patch, split changes on
> small parts (as was suggested by Thomas earlier) and post them
> one-by-one.
I think it would be really good if you could resend a complete new
patch that integrates your changes. You can credit Ismael for the
original work.
Best regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-03-18 9:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-14 19:46 [Buildroot] vlc: new package Алексей Бродкин
2013-03-18 9:31 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox