#13365 closed bug (fixed)
[Repositories] Haiku name should not be used explicitly
Reported by: | KapiX | Owned by: | perelandra |
---|---|---|---|
Priority: | normal | Milestone: | Unscheduled |
Component: | Preferences/Repositories | Version: | R1/Development |
Keywords: | gsoc2017 | Cc: | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
Given that Haiku can be built in non-official mode, Haiku name should not be used directly.
Here: http://cgit.haiku-os.org/haiku/tree/src/preferences/repositories/RepositoriesView.cpp#n154
Should be done like in Tracker: http://cgit.haiku-os.org/haiku/tree/src/kits/tracker/FSUtils.cpp#n735
Attachments (2)
Change History (20)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
While I don't mind changing the string, I don't think anybody is actually using that non-official mode currently, so I don't think we should dump any developer resources in this direction -- if anyone actually needs this, he could send in the patches himself :-)
comment:3 by , 8 years ago
Yeah, agreed. But here it's easiest to just leave out "Haiku" completely. It's unneeded, we don't say "Open with Haiku's Tracker" etc. either. :)
comment:4 by , 8 years ago
patch: | 0 → 1 |
---|
Here's the patch, replaced the string and removed the word 'Haiku', must be fine now! :) And there goes my first contribution.
follow-up: 6 comment:5 by , 8 years ago
There's still the "Select" in line 153 that has to be removed. Also "package" instead of "Packet" and remove the space in front of "Enable".
by , 8 years ago
Attachment: | 0001-String-modified.patch added |
---|
comment:6 by , 8 years ago
Replying to humdinger:
There's still the "Select" in line 153 that has to be removed. Also "package" instead of "Packet" and remove the space in front of "Enable".
I hope things are correct now with the new patch. Sorry for messing up.
comment:7 by , 8 years ago
For one, it's still lower case "package". Then the second patch relies on the 1st patch. You should create a new single patch to be applied to the orignal file. So revert by checking out the original RepositoriesView.cpp and make your changes once more. Dunno if there's a better solution, I'm not that familiar with the git intricacies...
comment:8 by , 8 years ago
Please break the long line up into two lines like the orignal code was. Please review the coding guidelines.
"A line must not have more than 80 columns; when wrapping a line, you usually indent one extra tab, but that can get more depending on the contents."
comment:9 by , 8 years ago
Still off by two spaces. "...you usually indent one extra tab, but that can get more depending on the contents."
Tab stops should be set to 4 spaces.
follow-up: 11 comment:10 by , 8 years ago
Read: replace the 2 spaces at the start of line 154 with one tab.
comment:11 by , 8 years ago
Replying to humdinger:
Read: replace the 2 spaces at the start of line 154 with one tab.
And finally done. Really sorry for the mess, won't happen in future. Understood the working. Let me know if there's any more changes.
follow-up: 13 comment:12 by , 8 years ago
Really sorry, I don't want to frustrate you... It's one tab, not 4 spaces.
comment:13 by , 8 years ago
Replying to humdinger:
Really sorry, I don't want to frustrate you... It's one tab, not 4 spaces.
Upon one backspace, the whole tab gets moved. It's one tab, I'm not sure why it shows 4 spaces for you. Please check now.
comment:14 by , 8 years ago
Really sorry, I don't want to frustrate you... It's one tab, not 4 spaces.
@scottmc, @humdinger: Trac displays tabs in patches as spaces. The actual patch file looks OK to me.
comment:15 by , 8 years ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:16 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
@waddlesplash: the original patch had those 4 spaces, anirudh has corrected since.
Applied with hrev51006. Thanks!
comment:17 by , 8 years ago
Marking this ticket with the keyword gsoc2017 to indicate it as one of my submission for Summer of Code 2017.
comment:18 by , 8 years ago
Keywords: | gsoc2017 added |
---|
Even simpler: "Enable the repositories to be used by package management:"