[jitsi-dev] [PATCH] Modify the targets 'pulseaudio' and 'opus' in src/native/build.xml in order to compile the libraries under FreeBSD


#1

Operating System: FreeBSD 9.1-RELEASE amd64
LibJitsi version: Latest source code snapshot in time of writing (commit f7b42e0fb7)

The attached patch modifies the targets 'pulseaudio' and 'opus' of the src/native/build.xml build file in order to produce the libjnpulseaudio.so and libjnopus.so shared libraries for FreeBSD under FreeBSD.
If you accept the patch, please also create the directories for FreeBSD under lib/native.
mkdir lib/native/freebsd lib/native/freebsd-64

I have used these 2 libraries in order to have an available Audio System in Jitsi under FreeBSD (we had none), and using the opus codec to perform a successful call.
In order PulseAudio to be available in the drop down menu of Audio System, I had to make minor modifications to the source files:
src/org/jitsi/impl/neomedia/device/DeviceConfiguration.java
src/org/jitsi/impl/neomedia/device/DeviceSystem.java
I am not sure yet if the modifications have any side effect, for this reason I didn't send them as a patch. For now I will keep them as patches for the Jitsi port of FreeBSD.

Regards

src-native-build.xml.diff (4.33 KB)

···

--
Tzanetos Balitsaris

----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.


#2

Hello,

···

On 8/26/13 5:04 PM, Tzanetos Balitsaris wrote:

Operating System: FreeBSD 9.1-RELEASE amd64
LibJitsi version: Latest source code snapshot in time of writing
(commit f7b42e0fb7)

The attached patch modifies the targets 'pulseaudio' and 'opus' of the
src/native/build.xml build file in order to produce the
libjnpulseaudio.so and libjnopus.so shared libraries for FreeBSD under
FreeBSD.
If you accept the patch, please also create the directories for
FreeBSD under lib/native.
mkdir lib/native/freebsd lib/native/freebsd-64

Thank you for your contribution! Before we can apply your patches, you
will need to sign our contributor agreement[1].

Regards,
Boris

[1] http://bluejimp.com/bca.pdf


#3

Hello,

Operating System: FreeBSD 9.1-RELEASE amd64
LibJitsi version: Latest source code snapshot in time of writing
(commit f7b42e0fb7)

The attached patch modifies the targets 'pulseaudio' and 'opus' of the
src/native/build.xml build file in order to produce the
libjnpulseaudio.so and libjnopus.so shared libraries for FreeBSD under
FreeBSD.

Your patch has been committed and ack'ed on the contributors page. Thank
you!

If you accept the patch, please also create the directories for
FreeBSD under lib/native.
mkdir lib/native/freebsd lib/native/freebsd-64

I created those separately, because I couldn't add an empty directory to
git, so I had to add a file in each.

Regards,
Boris

···

On 8/26/13 5:04 PM, Tzanetos Balitsaris wrote:


#4

Is there a reason not to do this in the build.xml?

--sent from my mobile

···

On 28 Aug 2013 15:41, "Boris Grozev" <boris@jitsi.org> wrote:

Hello,

On 8/26/13 5:04 PM, Tzanetos Balitsaris wrote:
> Operating System: FreeBSD 9.1-RELEASE amd64
> LibJitsi version: Latest source code snapshot in time of writing
> (commit f7b42e0fb7)
>
> The attached patch modifies the targets 'pulseaudio' and 'opus' of the
> src/native/build.xml build file in order to produce the
> libjnpulseaudio.so and libjnopus.so shared libraries for FreeBSD under
> FreeBSD.

Your patch has been committed and ack'ed on the contributors page. Thank
you!

> If you accept the patch, please also create the directories for
> FreeBSD under lib/native.
> mkdir lib/native/freebsd lib/native/freebsd-64

I created those separately, because I couldn't add an empty directory to
git, so I had to add a file in each.

Regards,
Boris

_______________________________________________
dev mailing list
dev@jitsi.org
Unsubscribe instructions and other list options:
http://lists.jitsi.org/mailman/listinfo/dev


#5

To not do what, create the directories? I don't see any reason not to do
it. I chose to add them to the repo, because it seemed like the most
straightforward thing to do.

Regards,
Boris

···

On 8/28/13 4:45 PM, Emil Ivov wrote:

Is there a reason not to do this in the build.xml?


#6

Quoting Boris Grozev <boris@jitsi.org>:

Hello,

Operating System: FreeBSD 9.1-RELEASE amd64
LibJitsi version: Latest source code snapshot in time of writing
(commit f7b42e0fb7)

The attached patch modifies the targets 'pulseaudio' and 'opus' of the
src/native/build.xml build file in order to produce the
libjnpulseaudio.so and libjnopus.so shared libraries for FreeBSD under
FreeBSD.

Your patch has been committed and ack'ed on the contributors page. Thank
you!

Thank you for reviewing it and committing it.

If you accept the patch, please also create the directories for
FreeBSD under lib/native.
mkdir lib/native/freebsd lib/native/freebsd-64

I created those separately, because I couldn't add an empty directory to
git, so I had to add a file in each.

Regards,
Boris

Regards

···

On 8/26/13 5:04 PM, Tzanetos Balitsaris wrote:

--
Tzanetos Balitsaris

----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.