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

Warning on apt-fast.config file while reconfigure apt-fast #102

Closed
Dab0u1 opened this issue Apr 14, 2016 · 4 comments
Closed

Warning on apt-fast.config file while reconfigure apt-fast #102

Dab0u1 opened this issue Apr 14, 2016 · 4 comments
Assignees

Comments

@Dab0u1
Copy link

Dab0u1 commented Apr 14, 2016

Hi,

Dont know if I am in the right place but...
When you run dpkg-reconfigure on apt-fast version 1.8.3137+git7b72bb7-0ubuntu1ppa3 (on willy and xenial) wich is on https://launchpad.net/~saiarcot895 ppa you have this warning at the end of the reconfiguring:
Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/${ <-- HERE _MAXNUM}/ at /var/lib/dpkg/info/apt-fast.config line 205."
The original line is:
if ($downloadcmd =~ /${_MAXNUM}/){
But it can be replaced by
if ($downloadcmd =~ /${_MAXNUM}/){ and the regex warning will disapear.

Edit: I'm sorry but I don't know why but the corrected code don't appear even in the brace, so I put space to see it:
if ($downloadcmd =~ /$ \ <-- HERE {_MAXNUM}/){

@Burnfaker
Copy link
Contributor

hi, can you put it in the code braces?

so here :)

@chrstphrchvz
Copy link
Contributor

I think this has to do with PPA/.deb packaging rather than the code on this repo, @saiarcot895 is who I would refer this to

@bepstein111
Copy link

bepstein111 commented Oct 22, 2017

So this has just always been a thing that I ignored while using apt-fast, but after trying to install today on Kubuntu 17.10, it's outright preventing me from using apt-fast at all. Maybe it's more strict about illegal regex? Anyway, while attempting to fix this, I've run into multiple issues:
First, the above "solution" is incorrect. I think it has the solution in the place of the original line:
if ($downloadcmd =~ /\${_MAXNUM}/){
(notice the /\ before ${_MAXNUM/})
and second, upon trying the solution, dpkg complains that _MAXNUM was never declared:
Global symbol "$_MAXNUM" requires explicit package name (did you forget to declare "my $_MAXNUM"?) at /var/lib/dpkg/info/apt-fast.config line 212.
It might not have to do with the code, as @chrstphrchvz said, but if it's going to break the whole app, then it's our problem.

@Lasall
Copy link
Collaborator

Lasall commented Oct 29, 2017

Thank you for reporting this issue and providing a fix. I updated the apt-fast/stable PPA with the current version and fixed this issue in the config file.

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

No branches or pull requests

5 participants