Missing the sed lines

Bug #583383 reported by LeoKomarov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uCareSystem
Fix Released
Medium
Salih EMIN

Bug Description

You're missing the "sed -i '/firstrun=1/d' <filename>" line from all of the install scripts, including depend_apt.
It has to go in on the line straight after the 1st "then".

Revision history for this message
Salih EMIN (salih-emin) wrote :

Do you me something like :

(from depend_apt.module)
## Installation to run on the first run of the script. ONLY on the first run of the script!
firstrun=0
firstrun=1
if [ $firstrun -gt 0 ]
then

   sed -i '/firstrun=1/d' <filename> # <----- YOURE Sugestion ?????

  if [ -f /usr/share/icons/hicolor/128x128/apps/2clickupdate_icon.png ]
   then
       echo "2clickupdate ICON found"
       echo ""
       sleep 1
   else
   ................... bla bla bla

Revision history for this message
LeoKomarov (leokomarov) wrote :

Yes, but change <filename> to depend_apt.module (in this case), and to 2clickUpdateCore for the cli version, etc.

Salih EMIN (salih-emin)
Changed in 2clickupdate:
importance: Undecided → Medium
assignee: nobody → Salih EMIN (salih-emin)
milestone: none → 5.7
Revision history for this message
Salih EMIN (salih-emin) wrote :

Fixed in 2cuCORE 5.7.

For 2ClickUpdate 5.7 this not a problem now as it is released in .deb package form that deals with any dependencies automatically.

Changed in 2clickupdate:
status: New → Fix Released
Revision history for this message
Salih EMIN (salih-emin) wrote :

As from now depend_apt module is obsolete and will be removed till version 6.0 of 2Click Update

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

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