Re: GitHub - alsa-project - repositories

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jaroslav,

On 2018/11/05 17:49, Jaroslav Kysela wrote:
Hi all,

   URL: https://github.com/alsa-project

   I finally finished the first phase of the integration with GitHub. All
repositories are now on GitHub and all repositories are mirrored to
alsa-project.org. The github repositories are master (developers should
push changes only to those repositories). If you refer the repository as
the code source (for packaging or so), please, keep to use the
repositories on alsa-project.org (in case when we migrate to other
service in future like GitLab or so). The mirroring is realtime, so the
changes should be visible on git.alsa-project.org in few seconds after
the push.

   I invited few people to the GitHub team and actually Takashi has full
access to all repos, Vinod Koul should have the write access to
tinycompress and Takashi Sakamoto should have the write access to
alsa-gi. If I omitted someone (or someone is not on github), please, let
me know (and register before).

   Because github adds possibility for the pull requests and issue
tracking, I added notifications for them to this (alsa-devel) mailing
list. The short notification should be sent when an pull request or an
issue is opened or changed. I think that it would be best to handle this
per request, so the developer can ask to resend the patch to the mailing
list for the wider review or just push the change with the signing.

   I activated Travis CI for alsa-lib and alsa-utils and I will add other
repos soon, too. URL: https://travis-ci.org/alsa-project

   If you have some ideas which other github applications can be used to
improve the code maintenance, let me know. I will probably play with the
coverity checker (http://scan.coverity.com), too.

Thanks for your arrangement. This is what I wish for a long time ;)
(however, I apologize my laziness forward to it...)

Just now I sent my PR to alsa-utils[1], and a notification is sent[2]. A
job of Travis-CI runs automatically on Ubuntu 14.04 LTS (probably in a
docker container)[3]. In the job, alsa-lib was firstly built, then
alsa-utils was built, as described in '.travis.yaml'. It takes a bit
time to download tex-related packages from us mirror of repository
(http://us-east-1.ec2.archive.ubuntu.com/). A few minutes later,
check icon becomes green on github.com.

I have two concerns.

1. For Ubuntu 14.04 LTS (trusty), End-of-life (EOL) is scheduled April
2019. A few months remained but it's better to use recent LTSs such as
18.04 (bionic) to reduce future maintenance cost. (I guess packages
except on 'main' pocket are not already maintained for security
updates.)

2. Message for PR
In my opinion, notification to alsa-devel list is an alternative of
cover-letters in the past. But in this time it doesn't includes change
summary, like:

```
Takashi Sakamoto (3):
  aplay: delete paragraph for obsoleted '--sleep-min' ('-s') option from
    aplay manual
  aplay: add a paragraph for '--samples' ('-s') option to aplay manual
  aplay: improve available conditions for '--samples' and '--duration'
    options

 aplay/aplay.1 | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)
```

If it's difficult to include the change summary automatically into the
notification, it's worth to discuss that PR senders should include it
handy to PR message.

[1] https://github.com/alsa-project/alsa-utils/pull/1
[2] http://mailman.alsa-project.org/pipermail/alsa-devel/2018-November/141505.html [3] https://travis-ci.org/alsa-project/alsa-utils/builds/451149201?utm_source=github_status&utm_medium=notification
[4] https://www.ubuntu.com/about/release-cycle


Thanks

Takashi Sakamoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux