Bug 239090 – Add PlayStation builds to bot watcher's dashboard.

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Ross Kirsling]

Add PlayStation builds to bot watcher's dashboard.

[Ross Kirsling]

Created attachment 457294 [details]
Patch

[Jonathan Bedard]

Comment on attachment 457294 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=457294&action=review

> Tools/CISupport/.gitignore:-8
> -public_html/

I think Aakash put this in quite deliberately because we end up with a bunch of logs in this directory on build.webkit.org.

[Ross Kirsling]

(In reply to Jonathan Bedard from comment #2)
> Comment on attachment 457294 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=457294&action=review
> 
> > Tools/CISupport/.gitignore:-8
> > -public_html/
> 
> I think Aakash put this in quite deliberately because we end up with a bunch
> of logs in this directory on build.webkit.org.

Okay, but it means that we can't add new files to the directory, so we'd need to replace it with something more specific.

[Darin Adler]

Comment on attachment 457294 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=457294&action=review

>>> Tools/CISupport/.gitignore:-8
>>> -public_html/
>> 
>> I think Aakash put this in quite deliberately because we end up with a bunch of logs in this directory on build.webkit.org.
> 
> Okay, but it means that we can't add new files to the directory, so we'd need to replace it with something more specific.

Are you sure that’s what it means? I’m pretty sure it just means git won’t complain when there are stray files; should not prevent doing a "git add" and committing the files.

[Ross Kirsling]

No, I mean it's not me performing cleanup; I was physically unable to create this patch without it:

> λ git add Tools/CISupport/build-webkit-org/public_html/dashboard/Images/PlayStation.png
> The following paths are ignored by one of your .gitignore files:
> Tools/CISupport/build-webkit-org/public_html
> hint: Use -f if you really want to add them.
> hint: Turn this message off by running
> hint: "git config advice.addIgnoredFile false"

[Ross Kirsling]

(In reply to Ross Kirsling from comment #5)
> No, I mean it's not me performing cleanup; I was physically unable to create
> this patch without it:
> 
> > λ git add Tools/CISupport/build-webkit-org/public_html/dashboard/Images/PlayStation.png
> > The following paths are ignored by one of your .gitignore files:
> > Tools/CISupport/build-webkit-org/public_html
> > hint: Use -f if you really want to add them.
> > hint: Turn this message off by running
> > hint: "git config advice.addIgnoredFile false"

Er, I guess that "hint" does work, so for the sake of this patch I can drop that change, but we definitely shouldn't being gitignoring committed files...the workaround here seems more like a bug than a feature to me. :-/

[Ross Kirsling]

Created attachment 457483 [details]
Patch

[Jonathan Bedard]

(In reply to Ross Kirsling from comment #6)
> (In reply to Ross Kirsling from comment #5)
> > No, I mean it's not me performing cleanup; I was physically unable to create
> > this patch without it:
> > 
> > > λ git add Tools/CISupport/build-webkit-org/public_html/dashboard/Images/PlayStation.png
> > > The following paths are ignored by one of your .gitignore files:
> > > Tools/CISupport/build-webkit-org/public_html
> > > hint: Use -f if you really want to add them.
> > > hint: Turn this message off by running
> > > hint: "git config advice.addIgnoredFile false"
> 
> Er, I guess that "hint" does work, so for the sake of this patch I can drop
> that change, but we definitely shouldn't being gitignoring committed
> files...the workaround here seems more like a bug than a feature to me. :-/

We should make a more narrow ignore, yeah, need to take a look on one of these bots to see exactly which files we're interested in ignoring.

[EWS]

Tools/Scripts/svn-apply failed to apply attachment 457483 [details] to trunk.
Please resolve the conflicts and upload a new patch.

[Ross Kirsling]

Created attachment 457564 [details]
Patch for landing

[Ross Kirsling]

Created attachment 457565 [details]
Patch for landing

[EWS]

Tools/Scripts/svn-apply failed to apply attachment 457565 [details] to trunk.
Please resolve the conflicts and upload a new patch.

[Ross Kirsling]

So yeah, this patch cannot be landed without fixing the .gitignore then.

[Ross Kirsling]

(In reply to Ross Kirsling from comment #13)
> So yeah, this patch cannot be landed without fixing the .gitignore then.

...landed via CQ, I should say. Going to try landing this via GitHub.

[EWS]

Committed r292843 (249616@main): <https://commits.webkit.org/249616@main>

Reviewed commits have been landed. Closing PR #284 and removing active labels.

[Radar WebKit Bug Importer]

<rdar://problem/91722149>