Page MenuHomePhabricator

efl_core_proc_env: add header that defined environ
AbandonedPublic

Authored by bu5hm4n on Feb 27 2019, 1:10 AM.

Details

Summary

ref T7710
Depends on D8037

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 9739
bu5hm4n created this revision.Feb 27 2019, 1:10 AM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

bu5hm4n requested review of this revision.Feb 27 2019, 1:10 AM
Peter2121 requested changes to this revision.Feb 27 2019, 12:59 PM

Nothing changed adding the unistd.h header.
Anyway, there is no

environ

variable declared in system's unistd.h.
I cannot find any header with such declaration.

This revision now requires changes to proceed.Feb 27 2019, 12:59 PM
bu5hm4n abandoned this revision.Mar 3 2019, 7:38 AM

libc5 appeared to have added the extern environ symbol to the header, so you don't have to add this blindly to every source file. Which is IMO the preferable solution, however, we have hammered it now across the tree. So whatever.