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

Upgrade to 6.2.57b.2 fails : dies in _load_include_admin_user_file() #1016

Closed
salaun-urennes1 opened this issue Oct 12, 2020 · 4 comments · Fixed by #1017
Closed

Upgrade to 6.2.57b.2 fails : dies in _load_include_admin_user_file() #1016

salaun-urennes1 opened this issue Oct 12, 2020 · 4 comments · Fixed by #1017
Labels
Milestone

Comments

@salaun-urennes1
Copy link
Collaborator

Sympa.pl --upgrade dies

Version

from 6.2.54 to 6.2.57b.2

Installation method

sources

Expected behavior

/usr/local/sympa/etc/data_structure.version should include new version number

Actual behavior

$ /usr/local/sympa/bin/sympa.pl --log_level=3 --upgrade
notice main:: Sympa 6.2.57b.2 Started
notice main:: Upgrade process...
notice main:: Upgrading from 6.2.54 to 6.2.57b.2...
notice Sympa::Upgrade::upgrade() Rebuilding config.bin files for ALL lists...it may take a while...
Use of uninitialized value in regexp compilation at /usr/local/sympa/bin/Sympa/List.pm line 5196, <GEN32> line 119.
err main::#872 > Sympa::Upgrade::upgrade#172 > Sympa::List::get_lists#4609 > Sympa::List::new#179 > Sympa::List::load#690 > Sympa::List::_load_list_config_file#5092 Expecting a paragraph for "creation" parameter in /usr/local/sympa/list_data/listes.univ-rennes1.fr/eco-secret-tech/config, ignore it
err main::#872 > Sympa::Upgrade::upgrade#172 > Sympa::List::get_lists#4609 > Sympa::List::new#179 > Sympa::List::load#690 > Sympa::List::_load_list_config_file#5092 Expecting a paragraph for "creation" parameter in /usr/local/sympa/list_data/listes.univ-rennes1.fr/ecobio/config, ignore it
notice Sympa::Upgrade::upgrade() Rebuilding the admin_table...it may take a while...
notice Sympa::Spindle::ProcessRequest::_twist() Processing Sympa::Request <action=include;[email protected];role=owner>
Can't use string ("/usr/local/sympa/etc/listes.etud"...) as a HASH ref while "strict refs" in use at /usr/local/sympa/bin/Sympa/List.pm line 4144.

Additional information

I checked the code that makes sympa.pl dies :

_load_include_admin_user_postprocess(\%include);

This piece of cade was updated 3 months ago

I tried to add traces in that code with no success

@racke
Copy link
Contributor

racke commented Oct 12, 2020

Can you provide the config for this include?

@salaun-urennes1
Copy link
Collaborator Author

Here is an anonymized version of droit-2021 list config file:
config.txt

And here is the owner_include data_source:
owners_editors.incl.txt

@ikedas ikedas added the bug label Oct 12, 2020
@ikedas ikedas added this to the 6.2.58 milestone Oct 12, 2020
@ikedas
Copy link
Member

ikedas commented Oct 13, 2020

@salaun-urennes1 , could you please try the PR above?

@salaun-urennes1
Copy link
Collaborator Author

Hi Soji,

I had another try upgrading with that line added and it succeeded :

$ /usr/local/sympa/bin/sympa.pl --log_level=3 --upgrade
notice main:: Sympa 6.2.57b.2 Started
notice main:: Upgrade process...
notice main:: Upgrading from 6.2.54 to 6.2.57b.2...
...
notice main:: Upgrade process finished

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

Successfully merging a pull request may close this issue.

3 participants