Page MenuHomePhabricator
Feed Advanced Search

Nov 8 2014

avivey added a comment to T6504: Create command line script to create new diffusion repositories.

https://secure.phabricator.com/conduit/method/repository.create/ is the conduit call to create a new remote-tracking repository (There used to be a separate Repository "application", hence the name).

Nov 8 2014, 10:21 PM · Conduit, Diffusion
chad added projects to T6504: Create command line script to create new diffusion repositories: Diffusion, Conduit.
Nov 8 2014, 9:36 PM · Conduit, Diffusion
chad added a comment to T6503: Import repository not successful.

Is your server out of memory? Also, what is your PHP's memory limit set to?

Nov 8 2014, 6:33 PM · Diffusion
neerajgangwar updated the task description for T6503: Import repository not successful.
Nov 8 2014, 6:12 PM · Diffusion
neerajgangwar created T6503: Import repository not successful.
Nov 8 2014, 6:11 PM · Diffusion
techdragon added a comment to T4245: Make repository URIs slightly prettier.

Just throwing my +1, agreeing with the stuff in T4694 and wanting a way to remove the need for call signs.

Nov 8 2014, 9:25 AM · Prioritized, Diffusion (v3), KDE, Restricted Project
btrahan added a project to T1130: Add Mercurial tag/bookmark support to Diffusion: Mercurial.
Nov 8 2014, 7:46 AM · Mercurial, Diffusion
btrahan added a project to T4416: Drop "-u" flag from `hg pull` in daemons?: Mercurial.
Nov 8 2014, 7:46 AM · Mercurial, Diffusion
btrahan added a project to T4255: SVN property changes on directories not shown in Diffusion diff views?: Subversion.
Nov 8 2014, 7:43 AM · Subversion, Diffusion
btrahan added a project to T3915: When only a subdirectory is tracked in SVN, don't link crumbs above the tracked subdirectory in browse views: Subversion.
Nov 8 2014, 7:43 AM · Subversion, Diffusion
btrahan added a comment to T6490: Unable to copy or move directories with Subversion.

Here's the best link I could find...

Nov 8 2014, 7:01 AM · Subversion, Diffusion
• yyang added a comment to T6490: Unable to copy or move directories with Subversion.

Thanks holmboe! I think we got exactly the same issue. But is there a chance that the svn hosting used to be correct, and broke by any of the recent versions?

Nov 8 2014, 6:43 AM · Subversion, Diffusion

Nov 7 2014

btrahan closed T6350: Diffusion spamming the feed/commits going weird as Resolved.

Alrighty folks - after your next upgrade this should be completely resolved.

Nov 7 2014, 10:38 PM · Support Impact, Feed, Diffusion
btrahan claimed T6107: "Reply to inline comment" shortcut broken in Diffusion.
Nov 7 2014, 10:34 PM · Support Impact, Diffusion
epriestley closed T1191: Comments are truncated at first non-base-plane character, a subtask of T6350: Diffusion spamming the feed/commits going weird, as Resolved.
Nov 7 2014, 10:26 PM · Support Impact, Feed, Diffusion
chad added a project to T6490: Unable to copy or move directories with Subversion: Subversion.
Nov 7 2014, 8:39 PM · Subversion, Diffusion
bkm71 added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

In CentOS "yum install mercurial" installs version 1.4 by default

Nov 7 2014, 5:55 PM · Diffusion
epriestley merged task T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'! into Restricted Maniphest Task.
Nov 7 2014, 5:30 PM · Diffusion
epriestley added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

You're running Mercurial 1.4, which is about 5 years old (released Nov, 2009). Update to a modern version of Mercurial.

Nov 7 2014, 5:30 PM · Diffusion
bkm71 added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

On the phabricator box:

Nov 7 2014, 5:28 PM · Diffusion
epriestley added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

Can you run this command on your server and show us the output?

Nov 7 2014, 5:28 PM · Diffusion
bkm71 added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

what environment information can you provide?

Nov 7 2014, 5:24 PM · Diffusion
bkm71 added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

what do you mean by "doing update from Mercurial" ...?

Go to Phabricator=>Diffusion=>Select Mercurial Repository=>Edit Repository=>click "update Now"

Nov 7 2014, 5:23 PM · Diffusion
btrahan added a comment to T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.

"I’m getting following error when doing update from Mercurial"

Nov 7 2014, 5:00 PM · Diffusion
bkm71 updated the task description for T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.
Nov 7 2014, 4:59 PM · Diffusion
bkm71 created T6492: Diffusion=> Mercurial update fails : STDERR abort: unknown revision 'head()'!.
Nov 7 2014, 4:55 PM · Diffusion
chad added projects to T6490: Unable to copy or move directories with Subversion: Diffusion, Arcanist.
Nov 7 2014, 3:48 PM · Subversion, Diffusion

Nov 6 2014

michaeloa added a comment to T6350: Diffusion spamming the feed/commits going weird.

Need a way to add tokens to comments. :)

Nov 6 2014, 11:23 PM · Support Impact, Feed, Diffusion
anton.vladimirov added a comment to T6486: Pending Differential Revisions isn't filtered by branch.

So maybe just reconsidering the location of this in the UI would help?

Nov 6 2014, 11:09 PM · Diffusion
epriestley added a subtask for T6486: Pending Differential Revisions isn't filtered by branch: T3462: In Differential, expose "featurename (branched from master)" via Conduit.
Nov 6 2014, 11:05 PM · Diffusion
epriestley triaged T6486: Pending Differential Revisions isn't filtered by branch as Wishlist priority.

I think we're going to convert this to "Recent Pending Differential Revisions" at a minimum, and only show stuff updated in the last couple of weeks. That should help with some cases of this, although not the specific case mentioned here (see T5658).

Nov 6 2014, 11:05 PM · Diffusion
chad added a project to T6486: Pending Differential Revisions isn't filtered by branch: Diffusion.
Nov 6 2014, 10:49 PM · Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

D10800 will resolve the last known bits here. Will close when it lands.

Nov 6 2014, 10:25 PM · Support Impact, Feed, Diffusion
hvr added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.

btw, found another such case in our history pushed by @thoughtpolice

Nov 6 2014, 9:33 PM · Haskell.org, Diffusion
btrahan added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.
./bin/phd log
Nov 6 2014, 9:32 PM · Haskell.org, Diffusion
chad renamed T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line from Differential doesn't recognize commit landing w/ `Signed-off-by:` line to Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.
Nov 6 2014, 9:32 PM · Haskell.org, Diffusion
hvr added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.
In T6483#82485, @chad wrote:

I don't recall all the history here, but I seem to recall --signoff was a feature added to git for reviewing linux kernel patches.

Nov 6 2014, 9:28 PM · Haskell.org, Diffusion
btrahan added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.

Does this reproduce at all?

Nov 6 2014, 4:36 PM · Haskell.org, Diffusion
epriestley added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.

Hrrm, I would have expected this to associate. We did some work here recently (T5693 / T3686) on the Differential side ("Why did this associate?") but don't currently expose a link on the Diffusion side to see the other half of the story ("Why didn't this associate?").

Nov 6 2014, 3:51 PM · Haskell.org, Diffusion
chad added a comment to T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line.

I don't recall all the history here, but I seem to recall --signoff was a feature added to git for reviewing linux kernel patches.

Nov 6 2014, 3:34 PM · Haskell.org, Diffusion
chad raised the priority of T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line from Low to Needs Triage.
Nov 6 2014, 3:29 PM · Haskell.org, Diffusion
chad triaged T6483: Diffusion doesn't recognize commit landing w/ `Signed-off-by:` line as Low priority.
Nov 6 2014, 3:28 PM · Haskell.org, Diffusion

Nov 5 2014

aik099 added a comment to T6477: Branch/Tags custom fields not populated for SVN commits.

Great, big thanks.

Nov 5 2014, 1:35 PM · Diffusion
epriestley added a comment to T6477: Branch/Tags custom fields not populated for SVN commits.
phabricator/ $ ./scripts/aphront/aphrontpath.php /D123
Nov 5 2014, 1:33 PM · Diffusion
aik099 added a comment to T6477: Branch/Tags custom fields not populated for SVN commits.

I understand. Then please tell me one thing (I hope that's not complex): how to determine which controller have processed an url?

Nov 5 2014, 1:32 PM · Diffusion
epriestley added a comment to T6477: Branch/Tags custom fields not populated for SVN commits.

Sorry, the answers to those questions are very complex and I don't have time to walk you through it since we aren't interested in bringing this upstream.

Nov 5 2014, 1:28 PM · Diffusion
aik099 updated subscribers of T6477: Branch/Tags custom fields not populated for SVN commits.

@epriestley , if I'm about to implement this in my local fork, could you please tell me:

Nov 5 2014, 1:27 PM · Diffusion
aik099 added a comment to T6477: Branch/Tags custom fields not populated for SVN commits.

Maybe you can do as Atlassian FishEye did: for each repository defined a regex to determine trunk/branch/tag (3 separate regexps). Then we can match path of 1st file in commit (hopefully nobody will do cross-branch/tag commits) and detect branch/tag.

Nov 5 2014, 1:06 PM · Diffusion
epriestley closed T6477: Branch/Tags custom fields not populated for SVN commits as Wontfix.

We don't plan to support inferring branches and tags from directory structure conventions in SVN.

Nov 5 2014, 12:34 PM · Diffusion
aik099 created T6477: Branch/Tags custom fields not populated for SVN commits.
Nov 5 2014, 11:58 AM · Diffusion

Nov 4 2014

chad updated subscribers of T3794: Rebuild Diffusion/Audit on top of CustomField infrastructure.
Nov 4 2014, 8:28 PM · Restricted Project, Custom Fields, Diffusion
chad merged task T6470: Support custom fields in commits into T3794: Rebuild Diffusion/Audit on top of CustomField infrastructure.
Nov 4 2014, 8:28 PM · Audit, Diffusion
aik099 created T6470: Support custom fields in commits.
Nov 4 2014, 7:35 PM · Audit, Diffusion
chad merged task T6466: Use "getMonogram" method in Diffusion into T5655: Standardize naming conventions.
Nov 4 2014, 3:00 PM · Diffusion
aik099 added a comment to T6468: Undepricate "diffusion.createcomment" Conduit API call.

As we've already discussed that task is far from it's implementation. Until then current API call works, why mark it as deprecated and prevent anybody willing to add comment to commit from seeing that call on /conduit/ page. I guess deprecation status only affects visibility of that API call, but it still continues to work.

Nov 4 2014, 1:20 PM · Conduit, Diffusion
epriestley closed T6468: Undepricate "diffusion.createcomment" Conduit API call as Wontfix.

See T5873.

Nov 4 2014, 1:15 PM · Conduit, Diffusion
aik099 added a project to T6468: Undepricate "diffusion.createcomment" Conduit API call: Conduit.
Nov 4 2014, 1:09 PM · Conduit, Diffusion
aik099 created T6468: Undepricate "diffusion.createcomment" Conduit API call.
Nov 4 2014, 1:09 PM · Conduit, Diffusion
aik099 created T6466: Use "getMonogram" method in Diffusion.
Nov 4 2014, 8:08 AM · Diffusion

Nov 3 2014

epriestley merged T6459: Removing stale commits into T5715: Diffusion commit count is incorrect after forced push.
Nov 3 2014, 1:38 PM · Restricted Project, Diffusion
chad removed a project from T5644: Add controls to prevent `pygmentize` from saturating system resources: Phabricator.
Nov 3 2014, 2:51 AM · Differential, Diffusion
chad closed T5630: Global Diffusion "Autoclose only" as Wontfix.

I don't think this is worth pursuing currently, given work involved.

Nov 3 2014, 2:46 AM · Phabricator, Diffusion
chad removed a project from T4842: Log when a user's SSH keys are used and what IP accessed them: Phabricator.
Nov 3 2014, 2:41 AM · Security, Auth, Diffusion
chad edited projects for T4027: Link commit header also to bugtraq url, added: Remarkup; removed Phabricator.
Nov 3 2014, 1:32 AM · Remarkup, Diffusion
chad removed a project from T4159: Exceptions occurring in Diffusion which result in PHP-FPM worker processes crashing: Phabricator.
Nov 3 2014, 1:00 AM · Diffusion

Oct 31 2014

aik099 added a comment to T5029: Fill in missing context for low-context diffs against tracked repositories.

+1 from me.

Oct 31 2014, 8:58 AM · KDE, Restricted Project, Diffusion, Differential

Oct 30 2014

btrahan updated the task description for T6350: Diffusion spamming the feed/commits going weird.
Oct 30 2014, 9:36 PM · Support Impact, Feed, Diffusion
btrahan updated subscribers of T6350: Diffusion spamming the feed/commits going weird.
Oct 30 2014, 9:34 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T2183: unify "table of contents" views between Differential and Diffusion.

T6426 is a specific complaint about the differences here, asking for the file count to be surfaced consistently.

Oct 30 2014, 3:58 PM · Owners, Diffusion, Differential
btrahan merged task T6426: Show total number of files in a revision into T2183: unify "table of contents" views between Differential and Diffusion.
Oct 30 2014, 3:57 PM · Diffusion, Badge Awarded, Differential
btrahan updated subscribers of T2183: unify "table of contents" views between Differential and Diffusion.
Oct 30 2014, 3:57 PM · Owners, Diffusion, Differential
chad triaged T6426: Show total number of files in a revision as Wishlist priority.
Oct 30 2014, 3:33 PM · Diffusion, Badge Awarded, Differential

Oct 29 2014

epriestley added a comment to T6414: Ability to put comments on current code.

Some specific notes on this:

Oct 29 2014, 3:30 PM · Diffusion
btrahan closed T6414: Ability to put comments on current code as Wontfix.

This seems pretty funky to me versus something like T6008 where you could just edit the code as you go, achieving the goal of easily adding "TODO" comments all over the code. I also think the amount of people browsing the code via diffusion versus looking at the actual code itself is such that you're much better off with the comments actually being part of the code. I also had my first waking nightmare about the user confusion as these comments pop up in Differential / Audit views versus something like T6008 where they'd be there in the code itself in a rather sensical fashion.

Oct 29 2014, 3:24 PM · Diffusion
aik099 added a comment to T6418: Support of "file://" protocol in "Repository Remote URI" field.

Luckily the existing repos, that are configured with file:// protocol are still working. I hope you won't be removing that.

Oct 29 2014, 1:02 PM · Diffusion
epriestley closed T6418: Support of "file://" protocol in "Repository Remote URI" field as Wontfix.

We removed this in June for security reasons. You can find some discussion in the June changelog:

Oct 29 2014, 12:45 PM · Diffusion
aik099 created T6418: Support of "file://" protocol in "Repository Remote URI" field.
Oct 29 2014, 12:03 PM · Diffusion

Oct 28 2014

avivey updated the task description for T6414: Ability to put comments on current code.
Oct 28 2014, 6:33 PM · Diffusion
avivey created T6414: Ability to put comments on current code.
Oct 28 2014, 6:33 PM · Diffusion
Avish created T6410: Differential: Allow collapsing and expanding folders in file tree view.
Oct 28 2014, 2:55 PM · Differential

Oct 27 2014

btrahan added a revision to T6350: Diffusion spamming the feed/commits going weird: Restricted Differential Revision.
Oct 27 2014, 7:11 PM · Support Impact, Feed, Diffusion
btrahan updated subscribers of T6350: Diffusion spamming the feed/commits going weird.
Oct 27 2014, 5:28 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

T6395 identified a new stack trace for empty commits.

Oct 27 2014, 5:25 PM · Support Impact, Feed, Diffusion
epriestley updated subscribers of T1374: Keyboard usability of the "associate task" dialog.
Oct 27 2014, 11:23 AM · Restricted Project, Maniphest, Differential

Oct 24 2014

chad added a comment to T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do.

Audit previously ran older code that was inconsistent with how every other application works. That had been fixed and new user preferences were added in support. We don't have any plans to offer admins additional defaults to set for users (are there any?). Event listeners already exist for things like registration and are infinitly more flexible.

Oct 24 2014, 4:01 PM · Diffusion, Audit
metellius added a comment to T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do.

Not sure I understand what you mean by saying it's running on modern infrastructure?

Oct 24 2014, 3:35 PM · Diffusion, Audit
chad closed T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do as Resolved.

Audit is now running on modern infrastructure and is consistent will all other Phabricator applications.

Oct 24 2014, 2:29 PM · Diffusion, Audit
metellius added a comment to T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do.

Don't think so, I just added them using the LDAP import page, and they work as normal accounts for people who need to do something in phabricator.

Oct 24 2014, 1:39 PM · Diffusion, Audit
cburroughs added a comment to T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do.

Are these disabled accounts receiving the email?

Oct 24 2014, 1:26 PM · Diffusion, Audit
metellius created T6387: People that have an account but are not actively using phabricator should not receive emails for every commit they do.
Oct 24 2014, 12:28 PM · Diffusion, Audit

Oct 21 2014

chad closed T6304: Git hosting not working as documented as Invalid.

I can't reproduce this, but we'll keep an eye on it.

Oct 21 2014, 6:56 PM · Documentation, Daemons, Diffusion
staticshock added a comment to T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.

/tmp being full probably doesn't apply to me, since i don't have a special mount-point for it, so it should be sharing in the bounty of ~28GB of free space on /dev/sda1. Someone did pull the plug on the machine, but that was probably about half a year ago, and I haven't noticed these errors until just now.

Oct 21 2014, 6:48 PM · Diffusion
btrahan added a revision to T6350: Diffusion spamming the feed/commits going weird: Restricted Differential Revision.
Oct 21 2014, 5:02 PM · Support Impact, Feed, Diffusion
btrahan added a project to T6350: Diffusion spamming the feed/commits going weird: Support Impact.

Yeah, I think publishing feed stories and sending emails from importing repositories is badsauce. Patch in a moment.

Oct 21 2014, 5:00 PM · Support Impact, Feed, Diffusion
michaeloa added a comment to T6350: Diffusion spamming the feed/commits going weird.

Incidentally, I just imported another repository, which resulted in the feed receiving a bunch of commit messages and one of the other developers on that repository receiving a ton of e-mails (I received nothing, although I also have a few commits in that repository).

Oct 21 2014, 11:27 AM · Support Impact, Feed, Diffusion
michaeloa added a comment to T6350: Diffusion spamming the feed/commits going weird.

Will do.

Oct 21 2014, 8:32 AM · Support Impact, Feed, Diffusion
epriestley added a comment to T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.

Yeah, this is something with MySQL and not really directly related to Phabricator. I'd try:

Oct 21 2014, 4:10 AM · Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

Alrighty folks - if you are still impacted by this, could you please update, restart daemons, and then run

Oct 21 2014, 12:54 AM · Support Impact, Feed, Diffusion
btrahan added a subtask for T6350: Diffusion spamming the feed/commits going weird: T1191: Comments are truncated at first non-base-plane character.
Oct 21 2014, 12:52 AM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

(rPfda0b086b565aab6f8486ac5a57585320d63f16d is the commit in question that is barfing with the character encoding problem for me.)

Oct 21 2014, 12:46 AM · Support Impact, Feed, Diffusion