Modify

Opened 7 years ago

Closed 7 years ago

#7577 closed defect (fixed)

[Patch] igmpproxy init script needs to be updated to utilize start-stop daemon

Reported by: dmitry@… Owned by: developers
Priority: normal Milestone: Backfire 10.03.1
Component: packages Version: Trunk
Keywords: Cc:

Description

I wasn't really sure what the best method is to submit patches. I reckon I'll try the good old ticket method as noone seems to have applied my patch which i submited via mailing list.

right now the igmpproxy init script needs to be forked, thats why it should use start-stop instead...

Attachments (2)

0001-igmpproxy-replace-init-script-with-a-working-one.diff (2.2 KB) - added by dmitry@… 7 years ago.
the diff
0001-igmpproxy-replace-init-script-with-a-working-one.patch (2.7 KB) - added by dmitry@… 7 years ago.
the git patch

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by dmitry@…

the diff

Changed 7 years ago by dmitry@…

the git patch

comment:1 Changed 7 years ago by dmitry@…

Hey guys, I really don't like nagging, but this fix is trivial. This is just a fix to the init-script which forks the process correctly. Please commit it!

Thanks!

comment:2 Changed 7 years ago by dmitry@…

Just to reiterate: IGMPproxy *could* be patched so that one can run it as a daemon, but as it stands today the init-script is broken, because you can't run IGMPproxy as a daemon.

IMHO the best alternative is to use start-stop until upstream daemonizes it, i think maintaining such a patched version of igmpproxy is not worth the work.

comment:3 Changed 7 years ago by nico

  • Milestone changed from Kamikaze to Backfire 10.03.1
  • Resolution set to fixed
  • Status changed from new to closed

Applied a slightly modified version [22268], thanks !

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.