Go to main content

Textpattern CMS support forum

You are not logged in. Register | Login | Help

#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

Offline

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

etc
Developer
Registered: 2010-11-11
Posts: 5,194
Website GitHub

Re: pcntl needed?

ponsukeen wrote #314189:

Just curious if it’s really needed

No, AFAIK, txp diagnostics is too zealous here.

Offline

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

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 11,453
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

Offline

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

etc
Developer
Registered: 2010-11-11
Posts: 5,194
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.

Offline

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

etc
Developer
Registered: 2010-11-11
Posts: 5,194
Website GitHub

Re: pcntl needed?

Offline

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

gaekwad
Server grease monkey
From: People's Republic of Cornwall
Registered: 2005-11-19
Posts: 4,263
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.)

Offline

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

Destry
Member
From: Haut-Rhin
Registered: 2004-08-04
Posts: 4,912
Website

Re: pcntl needed?

Welcome to the community, ponsukeen! Nice first post.

Offline

Board footer

Powered by FluxBB