Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update perl2python.pl #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Swati1910
Copy link

No description provided.

@Poikilos
Copy link

Poikilos commented May 22, 2020

@Swati1910 , can you explain this better? It says "add files via upload" then "Update perl2python.pl".

  • Did you update it from somewhere else?
  • Did you make changes (and what are they)?
  • Is perl2python.pl in this repo or your PR by a different author than you or felisPasseridae?
  • Why is it broken now due to a missing dependency on CGI which is a strange requirement other than for perl websites (see below)?
Can't locate CGI.pm in @INC (you may need to install the CGI module) (@INC contains: /usr/local/lib64/perl5/5.30 /usr/local/share/perl5/5.30 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5) at ./perl2python.pl line 5.
BEGIN failed--compilation aborted at ./perl2python.pl line 5.

(I can't find a package that is a clear match on Fedora:

sudo dnf search perl | grep cgi
Last metadata expiration check: 0:01:35 ago on Fri 22 May 2020 12:26:48 PM EDT.
perl-CGI-Compile.noarch : Compile .cgi scripts to a code reference like ModPerl::Registry
perl-FCGI-Client.noarch : Client library for the fastcgi protocol
perl-CGI-Application-Plugin-DevPopup.noarch : Runtime cgiapp info in a popup window

)
Please note why your version is better and note the package name for the new dependency in the README in your branch so the README change appears in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants