Last modified: 2013-12-19 19:18:16 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T60101, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58101 - Update favicon community.ico
Update favicon community.ico
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
wmf-deployment
All All
: Low enhancement (vote)
: ---
Assigned To: vladjohn2013
http://www.google-melange.com/gci/tas...
:
Depends on:
Blocks: 45036
  Show dependency treegraph
 
Reported: 2013-12-06 15:33 UTC by vladjohn2013
Modified: 2013-12-19 19:18 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Community.ico update (14.73 KB, patch)
2013-12-09 14:08 UTC, vladjohn2013
Details

Description vladjohn2013 2013-12-06 15:33:40 UTC
Create a favicon including 16x16, 32x32, and 48x48 versions of [URL of the high-resolution logo]. Substitute community.ico at operations/mediawiki-config/docroot/bits/favicon with the new version.
Comment 1 vladjohn2013 2013-12-06 15:37:53 UTC
A little problem, the .ico file on the favicom seems to be corrupted. I can not open it.
Comment 2 Quim Gil 2013-12-06 15:43:13 UTC
I cannot open it either. I guess the ico refers to 

https://commons.wikimedia.org/wiki/File:Wikimedia_Community_Logo.svg ?
Comment 3 Tomasz W. Kozlowski 2013-12-06 16:35:26 UTC
I have no problems opening the file, but yes, it can be created anew from the SVG version of the community logo (or an exported PNG, whichever you prefer) that Quim linked in comment 2.
Comment 4 vladjohn2013 2013-12-09 14:08:06 UTC
Created attachment 14029 [details]
Community.ico update
Comment 5 Tomasz W. Kozlowski 2013-12-09 14:10:24 UTC
Can you subbmit this attachment to Gerrit please?
Comment 6 Quim Gil 2013-12-09 19:32:26 UTC
Vlad John reported problems installing git-review, see the GCI task for details. I haven't used gerrit-review in Windows, neither Gerrit without gerrit-review. 

Another option that comes to mind is to submit the patch via https://github.com/wikimedia/operations-mediawiki-config

Any ideas? Committing the patch (as opposed to simply attach the file here is indeed part of the task (since GCI is about learning, and code review is an interesting skill to learn).
Comment 7 Quim Gil 2013-12-09 19:38:20 UTC
Ah wait. Vlad, you can use https://tools.wmflabs.org/gerrit-patch-uploader/
Comment 8 Tomasz W. Kozlowski 2013-12-09 19:59:12 UTC
(In reply to comment #6)

> Vlad John reported problems installing git-review, see the GCI task for
> details. I haven't used gerrit-review in Windows, neither Gerrit without
> gerrit-review. 

I haven't used Windows in a while, so can't comment on that, but it's possible to submit commits to Gerrit without git-review using https://www.mediawiki.org/wiki/Gerrit/Alternatives_to_git-review

> Another option that comes to mind is to submit the patch via
> https://github.com/wikimedia/operations-mediawiki-config

YuviPanda's SuchABot isn't enabled on this repo, so you'd have to submit all Github pull requests by hand, which is a terrible terrible "solution".
Comment 9 vladjohn2013 2013-12-10 18:42:30 UTC
I tried to upload the patch using this site: tools.wmflabs.org/gerrit-patch-uploader/   , but got this error:

Upload failed
Reason: Patch failed (is your patch in unified diff format, and does it patch apply cleanly to master?) (check log above for details)

How should I create the patch file, and in what format? (I tried both an .ico file and a .diff file)
Comment 10 Quim Gil 2013-12-10 19:26:02 UTC
Hi Vlad, as commented in the GCI task already: 

The error message tells you that you need to submit a "patch in unified diff format" that can be applied "cleanly to master". 

Please detail at the bug report the command line instructions you are using to generate the diff file and the replies you are getting for each step. This way we can tell you whether you are doing something wrong.
Comment 11 vladjohn2013 2013-12-10 19:40:05 UTC
I am using a program for windows called UnxUtils...
The command i write looks like this:

diff -u oldfile newfile > patchfile

But it says diff:command not found.

I also tried to use WinMerge, where I opened the .ico file from the site, and the .ico file I created, and then in the Tools menu, I used Create patch, but it said I coukd not create a patch from the binarry.
Comment 12 Quim Gil 2013-12-10 19:54:43 UTC
Please us git. GCI is about learning. If you want to be a developer you must learn git.  :)
Comment 13 Gerrit Notification Bot 2013-12-18 23:17:33 UTC
Change 102586 had a related patch set uploaded by RashiqAhmad:
Updated community.ico

https://gerrit.wikimedia.org/r/102586
Comment 14 Gerrit Notification Bot 2013-12-19 19:13:47 UTC
Change 102586 merged by jenkins-bot:
Updated community.ico

https://gerrit.wikimedia.org/r/102586

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links