Unnecessary dependency on csh

Bug #251972 reported by Sebastian Keller
6
Affects Status Importance Assigned to Milestone
brother-cups-wrapper-ac (Ubuntu)
Fix Released
Low
Saivann Carignan
brother-cups-wrapper-bh7 (Ubuntu)
Fix Released
Low
Saivann Carignan
brother-cups-wrapper-common (Ubuntu)
Fix Released
Low
Saivann Carignan
brother-cups-wrapper-laser (Ubuntu)
Fix Released
Low
Saivann Carignan
brother-cups-wrapper-laser1 (Ubuntu)
Fix Released
Low
Saivann Carignan
brother-cups-wrapper-mfc9420cn (Ubuntu)
Fix Released
Low
Saivann Carignan

Bug Description

The packages brother-cups-wrapper-bh7 and brother-cups-wrapper-common depend on the package "csh", however csh does not seem to be used in any of them. Neither does any reference to it show up in "grep csh -R ." in the source-package directory.

Revision history for this message
mmomjian (matthew-momjian) wrote :

not a bug, marking invalid. If it says it need csh, i'm sure it does.

Changed in brother-cups-wrapper-bh7:
status: New → Invalid
Revision history for this message
Saivann Carignan (oxmosys) wrote :

Thanks for your bug report. Unfortunately, brother packages really depends on csh because of some specific scripts (not all, but some). Brothers fault ;)

Revision history for this message
Sebastian Keller (skeller) wrote :

What does it need csh for? It is not used in any of the scripts as interpreter, not even the binary components contain the string "csh". Also csh has no dependencies except for libc, so it isn't used to pull in other dependencies. Neither does the csh package contain any file except for maybe csh itself, which could be usefull for a printer driver. And the original brother drivers did not depend on csh either, at least the ones for my model. But I also checked some of the scripts for other drivers as well and couldn't find any reference to csh.
Am I missing something or where does the csh-dependency come from? If you don't know where it comes from, please don't mark this bug as invalid.

Revision history for this message
Sebastian Keller (skeller) wrote :

Are any of the brother packages in the two packages I have mentioned? Because if they are not, couldn't the dependency be removed for these two packages?

Revision history for this message
Saivann Carignan (oxmosys) wrote :

After investigation, I confirm what Sebastian Keller said. There's actually only the -extra package which really needs to depend on csh. If after complete verifications, it is safe to remove the csh dependencies, we should do it.

Changed in brother-cups-wrapper-bh7:
assignee: nobody → saivann
importance: Undecided → Low
status: Invalid → Confirmed
Changed in brother-cups-wrapper-ac:
assignee: nobody → saivann
importance: Undecided → Low
status: New → Confirmed
Changed in brother-cups-wrapper-common:
assignee: nobody → saivann
importance: Undecided → Low
status: New → Confirmed
Changed in brother-cups-wrapper-laser1:
assignee: nobody → saivann
importance: Undecided → Low
status: New → Confirmed
Changed in brother-cups-wrapper-laser:
assignee: nobody → saivann
importance: Undecided → Low
status: New → Confirmed
Changed in brother-cups-wrapper-mfc9420cn:
assignee: nobody → saivann
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Saivann Carignan (oxmosys) wrote :

I'll take care to fix this bug before intrepid final release, thanks for finding this!

Changed in brother-cups-wrapper-ac:
status: Confirmed → In Progress
Changed in brother-cups-wrapper-bh7:
status: Confirmed → In Progress
Changed in brother-cups-wrapper-common:
status: Confirmed → In Progress
Changed in brother-cups-wrapper-laser1:
status: Confirmed → In Progress
Changed in brother-cups-wrapper-laser:
status: Confirmed → In Progress
Changed in brother-cups-wrapper-mfc9420cn:
status: Confirmed → In Progress
Revision history for this message
Saivann Carignan (oxmosys) wrote :

Till Kamppeter : Can you review and upload my debdiff to intrepid? I don't provide a debdiff for -laser package because I asked FriedChicken to fix it at the same time than bug 206365.

Revision history for this message
Saivann Carignan (oxmosys) wrote :
Revision history for this message
Saivann Carignan (oxmosys) wrote :
Revision history for this message
Saivann Carignan (oxmosys) wrote :
Revision history for this message
Saivann Carignan (oxmosys) wrote :
Revision history for this message
Saivann Carignan (oxmosys) wrote :
Revision history for this message
Till Kamppeter (till-kamppeter) wrote :

This is all OK. I will upload them into Intrepid. Thanks for the fix.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-ac - 1.0.0-7-0ubuntu5

---------------
brother-cups-wrapper-ac (1.0.0-7-0ubuntu5) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-ac:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-bh7 - 1.0.0-10-0ubuntu5

---------------
brother-cups-wrapper-bh7 (1.0.0-10-0ubuntu5) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-bh7:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-common - 1.0.0-10-0ubuntu5

---------------
brother-cups-wrapper-common (1.0.0-10-0ubuntu5) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-common:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-laser1 - 1.0.2-1-0ubuntu6

---------------
brother-cups-wrapper-laser1 (1.0.2-1-0ubuntu6) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-laser1:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-mfc9420cn - 1.0.0-1-0ubuntu5

---------------
brother-cups-wrapper-mfc9420cn (1.0.0-1-0ubuntu5) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-mfc9420cn:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package brother-cups-wrapper-laser - 2.0.1-2-0ubuntu5

---------------
brother-cups-wrapper-laser (2.0.1-2-0ubuntu5) intrepid; urgency=low

  * debian/control: Remove not necessary csh dependency. (LP: #251972)

 -- Saivann Carignan <email address hidden> Thu, 28 Aug 2008 13:23:57 -0400

Changed in brother-cups-wrapper-laser:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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