Skip to content

allow for case where homepage = None when generating the docs#4626

Merged
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:no_homepage
Sep 8, 2024
Merged

allow for case where homepage = None when generating the docs#4626
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:no_homepage

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Sep 7, 2024

If an easyconfig has homepage = (None) then we fail with

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/simon/work/easybuild-framework/easybuild/main.py", line 799, in <module>
    main_with_hooks()
  File "/home/simon/work/easybuild-framework/easybuild/main.py", line 785, in main_with_hooks
    main(args=args, prepared_cfg_data=(init_session_state, eb_go, cfg_settings))
  File "/home/simon/work/easybuild-framework/easybuild/main.py", line 686, in main
    print(list_software(output_format=options.output_format, detailed=options.list_software == 'detailed'))
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/simon/work/easybuild-framework/easybuild/tools/docs.py", line 809, in list_software
    if '%(' in info[key]:
       ^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

@boegel boegel added this to the 4.9.3 milestone Sep 8, 2024
@boegel boegel merged commit e0e04fb into easybuilders:develop Sep 8, 2024
@branfosj branfosj deleted the no_homepage branch September 8, 2024 08:24
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 this pull request may close these issues.

2 participants