Textpattern CMS

#1 2018-09-27 12:56:43

ponsukeen

New Member

From: Canada

Registered: 2018-09-27

Posts: 1

pcntl needed?

Just curious if it’s really needed, just reinstalled my vps and switched to php 5.4(And as always, I run nginx) and noticed this error on a fresh install.

The following PHP functions (which may be necessary to run Textpattern) are disabled on your server: pcntl_alarm, pcntl_fork, pcntl_waitpid, pcntl_wait, pcntl_wifexited, pcntl_wifstopped, pcntl_wifsignaled, pcntl_wexitstatus, pcntl_wtermsig, pcntl_wstopsig, pcntl_signal, pcntl_signal_dispatch, pcntl_get_last_error, pcntl_strerror, pcntl_sigprocmask, pcntl_sigwaitinfo, pcntl_sigtimedwait, pcntl_exec, pcntl_getpriority, pcntl_setpriority,

I remember reading that it’s bad to have it enabled on production servers(not that I’m hosting tons of large sites or that my site even gets hits) so I was hesitant to enable it. Thought I would ask before I did enable it because if it’s not really needed and it’s not really good to run on a production server then meh. :p

Regards

#2 2018-09-27 13:12:35

etc

Developer

Registered: 2010-11-11

Posts: 5,189

Website GitHub

Re: pcntl needed?

ponsukeen wrote #314189:

Just curious if it’s really needed

No, AFAIK, txp diagnostics is too zealous here.

---

bloatware

#3 2018-09-27 14:21:22

Bloke

Developer

From: Leeds, UK

Registered: 2006-01-29

Posts: 11,447

Website GitHub

Re: pcntl needed?

If we truly don’t need it – and honestly I’ve never seen a sniff of it in the code – then let’s just ditch the checks in Diagnostics.

---

The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.

Txp Builders – finely-crafted code, design and Txp

#4 2018-09-27 20:19:23

etc

Developer

Registered: 2010-11-11

Posts: 5,189

Website GitHub

Re: pcntl needed?

Bloke wrote #314192:

If we truly don’t need it – and honestly I’ve never seen a sniff of it in the code – then let’s just ditch the checks in Diagnostics.

The manual says

Process Control should not be enabled within a web server environment and unexpected results may happen if any Process Control functions are used within a web server environment.

So we shouldn’t rely on PCNTL anywhere in the code, me deduce. And Pete’s demo site runs without.

---

bloatware

#5 2018-09-27 20:29:40

etc

Developer

Registered: 2010-11-11

Posts: 5,189

Website GitHub

Re: pcntl needed?

Ditched

---

bloatware

#6 2018-10-01 19:20:22

gaekwad

Server grease monkey

From: People's Republic of Cornwall

Registered: 2005-11-19

Posts: 4,259

GitHub

Re: pcntl needed?

The pcntl_* stuff appears because it’s a part of the Ubuntu (and possibly Debian) PHP distribution.

etc wrote #314194:

And Pete’s demo site runs without.

(Actually, Pete just suppressed the warning…ahem…but don’t tell anyone.)

---

Textpattern CMS demo – tech updates

Code is Poetry; Text is Pattern.

#7 2018-10-22 08:54:44

Destry

Member

From: Haut-Rhin

Registered: 2004-08-04

Posts: 4,909

Website

Re: pcntl needed?

Welcome to the community, ponsukeen! Nice first post.