Closed Bug 1123017 Opened 9 years ago Closed 5 years ago

Avoid printing a trailing newline from milestone.py when running with a Windows python; r=gps

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → ehsan
Blocks: msvcwine
Why? Because now, what this does is that it doesn't print a newline at all. On any platform.

So what problem exactly are you trying to solve? That it outputs \r\n and you'd rather the \r not to be there?
Depends on: 1124906
Comment on attachment 8550827 [details] [diff] [review]
Avoid printing a trailing newline from milestone.py when running with a Windows python

Review of attachment 8550827 [details] [diff] [review]:
-----------------------------------------------------------------

Something is amiss here. Perl doesn't print newlines by default. `perl -e 'print "foo"' | hexdump -C`

What you should do is put all the prints in a new block with $\ set to undef. e.g.

{
  local $\ = undef;
  print(...);
}
Attachment #8550827 - Flags: review?(gps) → review-
Hmm, where does perl enter the picture here?  I'm not sure if I follow.
Flags: needinfo?(gps)
Ignore my stupidity. The flu has been ravaging me. You can do:

  print('foo', end='') to avoid printing a newline.
Flags: needinfo?(gps)
OK, thanks!  I will wait first for bug 1124906 to come to a resolution...
Product: Core → Firefox Build System
Assignee: ehsan → nobody
Assignee: nobody → ehsan
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: