p5-app-duckpan
                                
                                 p5-app-duckpan copied to clipboard
                                
                                    p5-app-duckpan copied to clipboard
                            
                            
                            
                        Add instructions for mac users if they failed to installl Crypt::SSLeay
Add docs for instructing mac os users to find their OpenSSL library when installing duckpan
I was actually in the process of removing the Crypt::SSLeay dependency as it should no longer be necessary:
https://github.com/duckduckgo/p5-duckpan-installer/pull/5
Are you interested in taking over that PR?
@moollaza funny you asked, I actually read https://github.com/duckduckgo/p5-duckpan-installer/pull/5/ before making this PR. I did fork the p5-duckpan-installer because I was hoping to simply the dockerfile. A couple of questions here.
- Any reason that we must run under non-root user?
- https://github.com/duckduckgo/p5-duckpan-installer/pull/5/ is almost half year old now, are there anything which is no longer applied to current duckPAN anymore?
- Is it safe to do https://github.com/soleo/p5-duckpan-installer/commit/e91767137fb5e3f729cfd7831d91b2c4298f1fe6 ? I added the mirror URL and notest flag to cpanm while installing.
- Any reason that we must run under non-root user?
I'm not sure why it restricts the root user -- this script was written several years ago and hasn't changed in the past 4 or 5.
@zachthompson do you have any objections to removing the non-root user requirement here?
- duckduckgo/p5-duckpan-installer#5/ is almost half year old now, are there anything which is no longer applied to current duckPAN anymore?
I suspect a lot of the dependencies we're adding here may no longer be used. Any Perl deps that are required should be added to DuckPAN's dist.ini so that installing via CPAN works the same as this script.
For example it appears Starman is used for duckpan publisher so we should probably add that dependency to the dist.ini since AutoReqs won't identify that as a requirement.
There's lots of room for improvement in this script, it just hasn't been a big issue and local DuckPAN installs are less popular now that we offer Codio.
- Is it safe to do soleo/p5-duckpan-installer@e917671 ? I added the mirror URL and notest flag to cpanm while installing.
I think we'd want to know up front about any errors when installing? If it's suppressed and the dev runs into issues, it'll be harder to know later what's going on
@moollaza probably not a good idea to run as root after I read http://stackoverflow.com/questions/4938592/how-why-does-npm-recommend-not-running-as-root/4999441.
Right, it's never a good idea to do development as root.
@zachthompson if you're running dev environment inside of a docker container, it's actually not a extremely bad idea. After all, anything happens inside of the container stays in container. 😄