781565 - ffprofile_{unix,win32}.py are misnamed and contain duplicate code

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Jeff Hammel]

http://hg.mozilla.org/build/talos/file/19fb848f813e/talos/ffprofile_win32.py
http://hg.mozilla.org/build/talos/file/19fb848f813e/talos/ffprofile_unix.py

So firstly this has nothing to firefox or profiles.  There is only one
function in each of these files, MakeDirectoryContentsWritable. There
is also high duplication between these functions, only the mode is
different, as well as high duplication between those in the
ffprocess*.py files.

This function should live in one place, either utils.py.  It should
determine the modeline from whether it is windows or unix.

This also should probably go in mozfile:
https://bugzilla.mozilla.org/show_bug.cgi?id=774916

Comment 1

[Jeff Hammel]

see also https://bugzilla.mozilla.org/show_bug.cgi?id=781324

Comment 2

[sam]

i'm looking to get involved with the mozilla community. anyone willing to help me get started on this bug, if not already assigned?

Comment 3

[Jeff Hammel]

(In reply to sam from comment #2)
> i'm looking to get involved with the mozilla community. anyone willing to
> help me get started on this bug, if not already assigned?

That would be me

Comment 4

[sam]

as in you are already working on it, or you are the mentor and need someone to take it on?

Comment 5

[Jeff Hammel]

Sorry, I should have clarified: I would be that mentor :)

Feel free to stop by irc://irc.mozilla.org/#ateam and say hi

Comment 6

[jdev005]

Attachment: any changes required ?

Comment 7

[Jeff Hammel]

Comment on attachment 652518 [details] [diff] [review]
any changes required ?

This looks good!

+  os_name=os.name
+  if os_name=='posix':
+    mod=0755
+  elif os_name=='nt':
+    mod=0777
+  else:
+   print('WARNING : this action is not supported on your current os')

It might be better to use mozinfo here, but probably not necessary.  (maybe?)

Comment 8

[Jeff Hammel]

pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=0e385d38a11d

Comment 9

[Jeff Hammel]

This doesn't quite work yet:

Traceback (most recent call last):
  File "run_tests.py", line 298, in <module>
    main()
  File "run_tests.py", line 295, in main
    run_tests(parser)
  File "run_tests.py", line 250, in run_tests
    talos_results.add(mytest.runTest(browser_config, test))
  File "/home/cltbld/talos-slave/talos-data/talos/ttest.py", line 262, in runTest
    profile_dir, temp_dir = self.createProfile(test_config['profile_path'], browser_config)
  File "/home/cltbld/talos-slave/talos-data/talos/ttest.py", line 132, in createProfile
    browser_config['webserver'])
  File "/home/cltbld/talos-slave/talos-data/talos/ffsetup.py", line 217, in CreateTempProfileDir
    self._hostproc.MakeDirectoryContentsWritable(profile_dir)
AttributeError: 'LinuxProcess' object has no attribute 'MakeDirectoryContentsWritable'
program finished with exit code 1
elapsedTime=0.748504

Comment 10

[Mozilla RelEng Bot]

Try run for 0e385d38a11d is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=0e385d38a11d
Results (out of 55 total builds):
    exception: 7
    success: 7
    failure: 41
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-0e385d38a11d

Comment 11

[jdev005]

Attachment: this should do it :)

Comment 12

[Jeff Hammel]

(In reply to jdev005 from comment #11)
> Created attachment 653080 [details] [diff] [review]
> this should do it :)

pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=b9da67ca43ad

Comment 13

[Mozilla RelEng Bot]

Try run for b9da67ca43ad is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=b9da67ca43ad
Results (out of 69 total builds):
    success: 62
    failure: 7
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-b9da67ca43ad

Comment 14

[Jeff Hammel]

(In reply to Mozilla RelEng Bot from comment #13)
> Try run for b9da67ca43ad is complete.
> Detailed breakdown of the results available here:
>     https://tbpl.mozilla.org/?tree=Try&rev=b9da67ca43ad
> Results (out of 69 total builds):
>     success: 62
>     failure: 7
> Builds (or logs if builds failed) available at:
> http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jhammel@mozilla.
> com-b9da67ca43ad

Looks very green! :)

Comment 15

[Jeff Hammel]

pushed: http://hg.mozilla.org/build/talos/rev/c986567c686a

Thanks for the patch!  I love to see useless code disappear