Ubuntu
chromium-browser package

[snap] Needlessly cats ~/.chromium-browser.init

Bug #1877593 reported by Roman Odaisky
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
chromium-browser (Ubuntu)
Fix Released
Low
Olivier Tilloy

Bug Description

$ fgrep -w cat -C4 /snap/chromium/current/bin/chromium.launcher

# Source ~/.chromium-browser.init for compatibility with the chromium-browser
# deb package (https://launchpad.net/bugs/1837746)
if [ -f $REALHOME/.chromium-browser.init ]; then
  cat $REALHOME/.chromium-browser.init 2>/dev/null
  if [ "$?" -eq "0" ]; then
    . $REALHOME/.chromium-browser.init
  fi
fi

This `cat` needlessly clutters the output when using things like `xdg-open`. If the intent is to check whether policies permit reading the file (so `test -f` wouldn’t work), then the stdout should be redirected to /dev/null as well.

Tags: snap
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report

The change was added in
https://git.launchpad.net/~chromium-team/chromium-browser/+git/snap-from-source/commit/?h=beta&id=5e1f049a

Redirecting the output makes sense

tags: added: snap
summary: - Needlessly cats ~/.chromium-browser.init
+ [snap] Needlessly cats ~/.chromium-browser.init
Revision history for this message
Olivier Tilloy (osomon) wrote :

Thanks for the report Roman. This should be fixed with https://git.launchpad.net/~chromium-team/chromium-browser/+git/snap-from-source/commit/?id=8a40b91c9e3817b8e01500b2152d11d373873193.

Changed in chromium-browser (Ubuntu):
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → Olivier Tilloy (osomon)
status: Confirmed → Fix Committed
Olivier Tilloy (osomon)
Changed in chromium-browser (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.