Actually pass "args" to modules

Bug #1090115 reported by bboe
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rubber
Fix Released
Medium
Sebastian Kapfer

Bug Description

I have a need to run dvips with both the arguments "-tletter -Pdownload35". Rubber appears to have support for args when using the `-o` and `-m`, however, close inspection of the source reveals that such options appear to be ignored.

The attached patch allows the passing of options to the dvips module when executed as:

    rubber -odvips:"-tletter -Pdownload35" -o ps2pdf paper.tex

While this patch produces the behavior that I desire, I am not familiar enough with rubber to ensure I didn't break anything in the process.

Revision history for this message
bboe (bboe) wrote :
Revision history for this message
Sebastian Kapfer (caci) wrote :

This is 1.3 material

Changed in rubber:
assignee: nobody → Sebastian Kapfer (caci)
importance: Undecided → Medium
milestone: none → 1.3
Revision history for this message
Sebastian Kapfer (caci) wrote :

This is not what do_options is for. It is intended to be invoked from a rubber directive like
% rubber: dvips.options -tletter -Pdownload35
in your paper.tex.

The -o dvips:quux indeed does nothing, and doesn't complain if passed options, which is a bug.

Changed in rubber:
status: New → Opinion
Revision history for this message
bboe (bboe) wrote :

Thanks for looking at this. Just to give you an update it would seem I figured out the `rubber directive` syntax which suited my needs a year ago. As I am no longer in academia I have little need for LaTeX and consequently rubber.

Sebastian Kapfer (caci)
Changed in rubber:
milestone: 1.3 → 1.4
Revision history for this message
Sebastian Kapfer (caci) wrote :

The original feature request was already solved (it's dvips.options for the record), now Rubber will also report invalid directives in a more useful way.

Thus closing this old bug, finally!

Changed in rubber:
status: Opinion → Fix Committed
Sebastian Kapfer (caci)
Changed in rubber:
milestone: 1.4 → none
status: Fix Committed → 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.