[GH-ISSUE #56] Ubuntu 17.10: chromium version is not parsed correctly #38

New issue

Closed

opened 2026-03-01 14:40:04 +03:00 by kerem · 2 comments

kerem commented

2026-03-01 14:40:04 +03:00

Owner

Originally created by @nasenpflaster on GitHub (Nov 28, 2017).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/56

Originally assigned to: @pirate on GitHub.

I get the following error when running archive.py:

[*] Checking Dependencies:
/usr/bin/chromium-browser
Traceback (most recent call last):
  File "./archive.py", line 124, in <module>
    update_archive(archive_path, links, source=source, resume=resume, append=True)
  File "./archive.py", line 70, in update_archive
    archive_links(archive_path, links, source=source, resume=resume)
  File "/home/benjamin/bin/bookmark-archiver/archive_methods.py", line 40, in archive_links
    check_dependencies()
  File "/home/benjamin/bin/bookmark-archiver/util.py", line 63, in check_dependencies
    if int(version) < 59:
ValueError: invalid literal for int() with base 10: 'Using'

the reason is that

chromium --version

might give something like

Using PPAPI flash.
--ppapi-flash-path=/usr/lib/adobe-flashplugin/libpepflashplayer.so --ppapi-flash-version=
Chromium 62.0.3202.94 Built on Ubuntu , running on Ubuntu 17.10

on Ubuntu 17.10. I changed line 62 of util.py

version = result.stdout.decode('utf-8').replace('Google Chrome ', '').replace('Chromium ', '').split(' ', 1)[0].split('.', 1)[0] # TODO: regex might be better
63

version = result.stdout.decode('utf-8').split('Chromium',1)[1].replace('Google Chrome ', '').replace('Chromium ', '').split(' ', 1)[1].split('.', 1)[0]

but could not find an immediate solution to also cover Google Chrome

Originally created by @nasenpflaster on GitHub (Nov 28, 2017). Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/56 Originally assigned to: @pirate on GitHub. I get the following error when running archive.py: ``` [*] Checking Dependencies: /usr/bin/chromium-browser Traceback (most recent call last): File "./archive.py", line 124, in <module> update_archive(archive_path, links, source=source, resume=resume, append=True) File "./archive.py", line 70, in update_archive archive_links(archive_path, links, source=source, resume=resume) File "/home/benjamin/bin/bookmark-archiver/archive_methods.py", line 40, in archive_links check_dependencies() File "/home/benjamin/bin/bookmark-archiver/util.py", line 63, in check_dependencies if int(version) < 59: ValueError: invalid literal for int() with base 10: 'Using' ``` the reason is that > chromium --version might give something like > Using PPAPI flash. > --ppapi-flash-path=/usr/lib/adobe-flashplugin/libpepflashplayer.so --ppapi-flash-version= > Chromium 62.0.3202.94 Built on Ubuntu , running on Ubuntu 17.10 on Ubuntu 17.10. I changed line 62 of util.py > version = result.stdout.decode('utf-8').replace('Google Chrome ', '').replace('Chromium ', '').split(' ', 1)[0].split('.', 1)[0] # TODO: regex might be better 63 to > version = result.stdout.decode('utf-8').split('Chromium',1)[1].replace('Google Chrome ', '').replace('Chromium ', '').split(' ', 1)[1].split('.', 1)[0] but could not find an immediate solution to also cover Google Chrome

kerem closed this issue

2026-03-01 14:40:04 +03:00

kerem commented

2026-03-01 14:40:05 +03:00

Author

Owner

@pirate commented on GitHub (Nov 28, 2017):

Ah, I knew trying to parse a version string was going to be trouble eventually :)

Fixed in 6eb1e23, try pulling master and running it again.

@pirate commented on GitHub (Nov 28, 2017): Ah, I knew trying to parse a version string was going to be trouble eventually :) Fixed in 6eb1e23, try pulling master and running it again.

kerem commented

2026-03-01 14:40:05 +03:00

Author

Owner

@pirate commented on GitHub (Nov 28, 2017):

Comment back if you're still experiencing problems and I'll reopen the issue.

@pirate commented on GitHub (Nov 28, 2017): Comment back if you're still experiencing problems and I'll reopen the issue.

kerem referenced this issue

2026-03-01 14:48:23 +03:00

[PR #200] [MERGED] Exclude blacklisted URLs #1089

kerem referenced this issue

2026-03-01 17:51:32 +03:00

[GH-ISSUE #38] Config: Add a way to blacklist URLs from being archived or indexed #1535

kerem referenced this issue

2026-03-01 18:00:05 +03:00

[PR #200] [MERGED] Exclude blacklisted URLs #2602

kerem referenced this issue

2026-03-14 20:45:29 +03:00

[GH-ISSUE #38] Config: Add a way to blacklist URLs from being archived or indexed #3045