Bug 238869 - www/chromium: sndio input implementation
Summary: www/chromium: sndio input implementation
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: amd64 Any
: --- Affects Many People
Assignee: Carlos J. Puga Medina
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-28 19:39 UTC by zielonka michal
Modified: 2019-07-17 18:47 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (chromium)


Attachments
sndio_input.cc sndio_input.h (8.72 KB, patch)
2019-06-28 19:39 UTC, zielonka michal
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zielonka michal 2019-06-28 19:39:54 UTC
Created attachment 205403 [details]
sndio_input.cc sndio_input.h

I had a problem with missing implementation for recording audio for sndiod enabled in chromium port.

The alsa option often had made for me problem with lost audio input. I've checked if there is a proposal for filling implementation for sndio input, but unfortunately I couldn't find it.

So I tried to make it myself, and the patch with working for me version I attached here.

/u/ports# git remote  get-url origin
https://github.com/trueos/trueos-ports
/u/ports# git branch 
* trueos-master
Comment 1 Tobias Kortkamp freebsd_committer 2019-06-29 03:16:23 UTC
OpenBSD has imported an input implementation a couple of days ago:
https://github.com/openbsd/ports/commit/a82a036f9349ae5e0eec05738cdd2c570ff5dd35

So we would get that by syncing with them.  A patch that applies it
against the FreeBSD ports tree is available here:
https://github.com/t6/freebsd-ports-sndio/commit/f213ba25a3460ed6f8e858f04f5592fca8edb7d8.patch
Comment 2 zielonka michal 2019-06-29 21:03:13 UTC
I was to slow :). I agree that their implementation looks good. But I added also setting volume which is missed there.

 Do you think that I can make rebase to PR to T6?

 Also I have question, will it be better commit changes by github and PRs? I'm waiting now for account in phabricator from several days. But I'm a truly little confused which way is better: mailing list, bugzilla, phabricator or github :)
Comment 3 Tobias Kortkamp freebsd_committer 2019-06-30 03:38:16 UTC
(In reply to zielonka michal from comment #2)
> I was to slow :). I agree that their implementation looks good. But I added
> also setting volume which is missed there.
> 
>  Do you think that I can make rebase to PR to T6?

I would suggest to rebase and then send your changes to the OpenBSD
devs (i.e., ratchov@openbsd.org and robert@openbsd.org) to review
and integrate.

>  Also I have question, will it be better commit changes by github and PRs?
> I'm waiting now for account in phabricator from several days. But I'm a
> truly little confused which way is better: mailing list, bugzilla,
> phabricator or github :)

IMHO Bugzilla is the first choice.  The other things should ideally
be accompanied by bugs on Bugzilla for tracking progress.
Comment 4 zielonka michal 2019-06-30 05:17:24 UTC
thanks and I close this bug :)
Comment 5 Tobias Kortkamp freebsd_committer 2019-06-30 06:25:23 UTC
(In reply to zielonka michal from comment #4)
> thanks and I close this bug :)

It should stay open for tracking of the issue until this is resolved by us
actually importing an sndio input implementation into our ports tree.
Comment 6 Rene Ladan freebsd_committer 2019-07-08 19:41:58 UTC
I merged this into into the GitHub branch, see https://github.com/gliaskos/freebsd-chromium/pull/157
Comment 7 commit-hook freebsd_committer 2019-07-09 08:21:47 UTC
A commit references this bug:

Author: cpm
Date: Tue Jul  9 08:20:57 UTC 2019
New revision: 506266
URL: https://svnweb.freebsd.org/changeset/ports/506266

Log:
  www/chromium: Add support for audio recording using sndio

  PR:		238869
  Submitted by:	Zielonka Michal <zielonka.michal@gmail.com>, tobik
  Obtained from:	https://github.com/t6/freebsd-ports-sndio/commit/f213ba25a3460ed6f8e858f04f5592fca8edb7d8
  MFH:		2019Q3

Changes:
  head/www/chromium/files/sndio_input.cc
  head/www/chromium/files/sndio_input.h
  head/www/chromium/files/sndio_output.cc
  head/www/chromium/files/sndio_output.h
Comment 8 commit-hook freebsd_committer 2019-07-17 18:47:13 UTC
A commit references this bug:

Author: cpm
Date: Wed Jul 17 18:47:03 UTC 2019
New revision: 506812
URL: https://svnweb.freebsd.org/changeset/ports/506812

Log:
  MFH: r506266

  www/chromium: Add support for audio recording using sndio

  PR:		238869
  Submitted by:	Zielonka Michal <zielonka.michal@gmail.com>, tobik
  Obtained from:	https://github.com/t6/freebsd-ports-sndio/commit/f213ba25a3460ed6f8e858f04f5592fca8edb7d8

  Approved by:	ports-secteam (joneum)

Changes:
_U  branches/2019Q3/
  branches/2019Q3/www/chromium/files/sndio_input.cc
  branches/2019Q3/www/chromium/files/sndio_input.h
  branches/2019Q3/www/chromium/files/sndio_output.cc
  branches/2019Q3/www/chromium/files/sndio_output.h