Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2a9a8fd0-356c-351d-612d-489a0352e1e7@adelielinux.org>
Date: Sat, 15 Dec 2018 19:34:12 -0600
From: "A. Wilcox" <awilfox@...lielinux.org>
To: musl@...ts.openwall.com
Subject: Re: additional patches for musl

On 12/15/18 18:01, сергей волковичь wrote:
> cabulertion.
> i finded additional patches for implementing some missing musl features.
> 
> 
> -- реклама -----------------------------------------------------------
> Поторопись зарегистрировать самый короткий почтовый адрес @i.ua
> https://mail.i.ua/reg - и получи 1Gb для хранения писем
> 

Hi there.

First of all, welcome and thank you for sending your patches.

I think the canonicalize_file_name and nftw stuff may be okay, though it
does add some bloat.  The qsort_r code may be as well.  These should be
looked at by the musl maintainers themselves to determine suitability
and issues.  (I know the nftw stuff would be useful for a few packages,
myself.)

The utmpx patch should probably be merged.

The printf stuff is LGPL licensed while musl is MIT.  That won't be
workable because the licenses are incompatible.

The strndupa patch does not give an alternative for non-GNU C compilers
and it seems rather arbitrary and gratuitous.  It is better to make the
software more portable than add this, IMO.

The glob and regex macros are going to break code and those are a firm
NAK from me, though I don't know how others feel about them.

Hope this helps.

Best,
--arw
-- 
A. Wilcox (awilfox)
Project Lead, Adélie Linux
https://www.adelielinux.org



Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.