Page MenuHomePhabricator
Feed All Stories

Apr 4 2016

eadler added a comment to T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust).

for future people: git seems to ignore ~/.curlrc but you can pass a proxy using -chttp.proxy or using the https_proxy variable.

Apr 4 2016, 9:26 PM · Prioritized, Restricted Project, Feature Request
eadler updated the task description for T10227: Provide an extension point for HTTP behaviors (proxies, SSL trust).
Apr 4 2016, 8:33 PM · Prioritized, Restricted Project, Feature Request
psawant added a comment to Q354: Svn imported repository diff does not show up.

Thanks you for looking into it. I was able to sync svn repositories and view the diff.

Apr 4 2016, 8:16 PM
lpriestley added a comment to T10671: Implement `badges.search` and `badges.edit` Conduit API methods.

Yeah, I was pondering the award/revoke flow. PasteEditConduitAPIMethod is so bare bones as an example, and PhabricatorBadgesEditEngine is so different from other edit engines... I'm just not sure how that will hook into the badges edit API method.

Apr 4 2016, 7:23 PM · Restricted Project, Conduit, Badges (v0)
epriestley added a comment to T10671: Implement `badges.search` and `badges.edit` Conduit API methods.

The new *.edit endpoints can both create and edit, so a badges.create method would be redundant.

Apr 4 2016, 7:20 PM · Restricted Project, Conduit, Badges (v0)
lpriestley added a comment to T10671: Implement `badges.search` and `badges.edit` Conduit API methods.

Is there a reason, other than scope, why we're not implementing a badges.create Conduit API method? Or any of the other ones?

Apr 4 2016, 7:17 PM · Restricted Project, Conduit, Badges (v0)
epriestley added a revision to T7303: Provide OAuth access to Conduit: Unknown Object (Differential Revision).
Apr 4 2016, 6:05 PM · Restricted Project, Conduit, Restricted Project
epriestley updated subscribers of T10721: SVN repositories with no callsigns do not route individual commit URLs properly.

@psawant, thanks for the report and for your help building a reproduction case for us on a Phacility instance. It was extremely helpful in identifying and resolving the root cause of this issue.

Apr 4 2016, 4:46 PM · Diffusion (v3)
epriestley closed T10721: SVN repositories with no callsigns do not route individual commit URLs properly, a subtask of T4245: Make repository URIs slightly prettier, as Resolved.
Apr 4 2016, 4:44 PM · Prioritized, Diffusion (v3), KDE, Restricted Project
epriestley closed T10721: SVN repositories with no callsigns do not route individual commit URLs properly as Resolved by committing Restricted Diffusion Commit.
Apr 4 2016, 4:44 PM · Diffusion (v3)
epriestley edited this Badge.
Apr 4 2016, 4:44 PM
epriestley added a revision to T10721: SVN repositories with no callsigns do not route individual commit URLs properly: Unknown Object (Differential Revision).
Apr 4 2016, 4:43 PM · Diffusion (v3)
chad added a comment to Q354: Svn imported repository diff does not show up.

Thanks for helping us out @psawant!

Apr 4 2016, 4:33 PM
epriestley added a comment to T10721: SVN repositories with no callsigns do not route individual commit URLs properly.

A workaround is to give the repository a callsign Edit Repository → Edit Basic Information → Callsign.

Apr 4 2016, 4:32 PM · Diffusion (v3)
epriestley changed the edit policy for T10721: SVN repositories with no callsigns do not route individual commit URLs properly.
Apr 4 2016, 4:31 PM · Diffusion (v3)
epriestley added a comment to Q354: Svn imported repository diff does not show up.

This is a bug; see T10721 for followup and discussion.

Apr 4 2016, 4:31 PM
epriestley created T10721: SVN repositories with no callsigns do not route individual commit URLs properly.
Apr 4 2016, 4:31 PM · Diffusion (v3)
epriestley added a comment to Q354: Svn imported repository diff does not show up.

Thanks, looking at this now.

Apr 4 2016, 4:25 PM
psawant added a comment to Q354: Svn imported repository diff does not show up.

i have restored the repository at http://ec2-52-49-69-145.eu-west-1.compute.amazonaws.com/svn/devpoets/

Apr 4 2016, 4:23 PM
epriestley added a comment to Q354: Svn imported repository diff does not show up.

I have taken down the public repository for time being.

Apr 4 2016, 4:10 PM
psawant added a comment to Q354: Svn imported repository diff does not show up.

the test instance is at test-7japztksvma2.phacility.com

Apr 4 2016, 3:13 PM
chad added a comment to Q354: Svn imported repository diff does not show up.

What's the Phacility test instance? Do you mind if we look at the error there?

Apr 4 2016, 3:09 PM
psawant added a comment to Q354: Svn imported repository diff does not show up.

I tried out phacility and still seeing the same error. I am going to try out creating a new SVN repository and import it. Will also be trying out a git repository.

Apr 4 2016, 3:06 PM
• Instinct64 added a watcher for Releeph: • Instinct64.
Apr 4 2016, 2:20 PM
epriestley added a revision to T7303: Provide OAuth access to Conduit: Unknown Object (Differential Revision).
Apr 4 2016, 2:15 PM · Restricted Project, Conduit, Restricted Project
epriestley merged task T10720: Phriction wiki page TOC does not represent the hierarchical structure of the document into T9868: PHUIDocumentViewPro ToC menu improvements.
Apr 4 2016, 1:15 PM · Bug Report
epriestley updated subscribers of T9868: PHUIDocumentViewPro ToC menu improvements.
Apr 4 2016, 1:15 PM · PHUI
epriestley added a comment to T10720: Phriction wiki page TOC does not represent the hierarchical structure of the document.

This report does not describe a bug.

Apr 4 2016, 1:15 PM · Bug Report
urzds created T10720: Phriction wiki page TOC does not represent the hierarchical structure of the document.
Apr 4 2016, 1:09 PM · Bug Report
epriestley added a comment to T9456: Evaluate upstream support for third-party build systems.

The staging area is a Git remote that we'll push to, so it should be a valid target for git push <url> ....

Apr 4 2016, 12:04 PM · Restricted Project, Harbormaster
epriestley added a comment to T10719: "status" field should not be copied from template when creating new tasks.

This is intentional, so you can use a task as a template to create other tasks if you have, say, several different "open" statuses. The "Create Subtask" action specifically forces the status to the default open status, but I think the current behavior is correct for bare ?template= which has been entered manually.

Apr 4 2016, 11:59 AM · Maniphest, Feature Request
• Marty updated Q356: Installation fails: Failed while trying to read schema status from to Installation fails: Failed while trying to read schema status.
Apr 4 2016, 11:56 AM
epriestley closed T10718: Please add an option to arc diff or arc land that can create revisions only and not push as Invalid.

@paladox, you've been warned repeatedly about your consistently low-quality interactions with this upstream.

Apr 4 2016, 11:52 AM · Arcanist, Feature Request
appplemac added a comment to T9456: Evaluate upstream support for third-party build systems.

@epriestley Configuring a test project with Phabricator and CircleCI, cannot understand what format should the staging area be in.

Apr 4 2016, 11:52 AM · Restricted Project, Harbormaster
Pawka added a revision to T10719: "status" field should not be copied from template when creating new tasks: Unknown Object (Differential Revision).
Apr 4 2016, 11:36 AM · Maniphest, Feature Request
• paladox added a comment to T10717: arc land does not work on windows.

@chad @epriestley arcanist works straight of the master branch in cmd on windows for me. But git for windows does not work.

Apr 4 2016, 11:30 AM · Windows, Bug Report (Needs Information), Arcanist
• paladox updated subscribers of T10717: arc land does not work on windows.
Apr 4 2016, 11:22 AM · Windows, Bug Report (Needs Information), Arcanist
• paladox updated subscribers of T10717: arc land does not work on windows.

@hach-que thanks for your suggestion, you helped me. I get different directors from different command line so cmd throughs C:\Program Files\Git\cmd\git.ex whereas git for windows throughs /mingw64/bin/git

Apr 4 2016, 11:22 AM · Windows, Bug Report (Needs Information), Arcanist
Pawka created T10719: "status" field should not be copied from template when creating new tasks.
Apr 4 2016, 11:21 AM · Maniphest, Feature Request
• paladox created T10718: Please add an option to arc diff or arc land that can create revisions only and not push.
Apr 4 2016, 11:15 AM · Arcanist, Feature Request
sascha-egerer awarded M1461: Timeline / Remarkup Cleanup a Love token.
Apr 4 2016, 8:35 AM
hach-que closed T6454: Implement support for KVM-based hosts in Drydock as Wontfix.

Closing this off as I don't use dynamically allocated hosts any more (it's too much maintenance of custom patches).

Apr 4 2016, 2:12 AM · Drydock
hach-que awarded T10697: Move Aphlict server configuration to a separate configuration file a Like token.
Apr 4 2016, 2:09 AM · Restricted Project, Aphlict
hach-que added a comment to T10717: arc land does not work on windows.

I'm hesitant to provide further discussion on this issue because of conversation above, but never-the-less in case anyone else using Windows stumbles across this issue, then here's the following advice:

Apr 4 2016, 2:06 AM · Windows, Bug Report (Needs Information), Arcanist

Apr 3 2016

• paladox added a comment to T10717: arc land does not work on windows.
In T10717#168288, @chad wrote:

I tried to reproduce everything. I have never personally developed on Windows, so everything was done from scratch.

  • Windows 10
  • Installed git from https://git-scm.com/download/win (2.8.0)
  • Installed PHP from http://windows.php.net/download/ (5.6.20, Non thread safe)
  • Installed phabricator, arcanist, libphutil from master/HEAD over ssh
  • Add PHP/Arcanist to global paths
  • Un-comment libcurl in php.ini
  • Install arcanist certificate
  • use arc patch to pull down patch (see D15596)
  • use arc land to push to master (see 7694a6729f6a)
Apr 3 2016, 11:09 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Or keep the patch locally yourself, that's fine as well.

Apr 3 2016, 10:58 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

That is, you are still required to provide a valid bug report (steps to reproduce) against a supported environment so that we may observe the bug and verify any fix.

Apr 3 2016, 10:56 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

And you understand any "fix" against a beta/unsupported environment is not valid in the upstream, correct?

Apr 3 2016, 10:55 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

Yes. It seems it may not be able to find the git. Since git for windows updated a lot of things so I doint know if that check works now. But also I doint know if your check will find git in the 64-bit folder or 32-bit.

Apr 3 2016, 10:52 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Are you still working on a beta of Windows?

Apr 3 2016, 10:48 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@epriestley and @chad ive done some digging at it seems to be this

Apr 3 2016, 10:41 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

I'm also not trying to waste your time.

Apr 3 2016, 10:07 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

You did not test the patch. As I noted, the patch runs git fetch --queit instead of git fetch --quiet.

no.png (222×595 px, 32 KB)

You also ignored the large explanatory comment immediately above the changed code which explains why it uses passthru and the specific problem that passthru solves, with a reference to the original issue.

Apr 3 2016, 10:02 PM · Windows, Bug Report (Needs Information), Arcanist
• eduard44 awarded T10538: Bridge GitHub to Phabricator a Love token.
Apr 3 2016, 9:52 PM · Nuance, GitHub
• paladox added a comment to T10717: arc land does not work on windows.

You did not test the patch. As I noted, the patch runs git fetch --queit instead of git fetch --quiet.

no.png (222×595 px, 32 KB)

You also ignored the large explanatory comment immediately above the changed code which explains why it uses passthru and the specific problem that passthru solves, with a reference to the original issue.

Apr 3 2016, 9:38 PM · Windows, Bug Report (Needs Information), Arcanist
epriestley added a comment to T10717: arc land does not work on windows.

You did not test the patch. As I noted, the patch runs git fetch --queit instead of git fetch --quiet.

Apr 3 2016, 8:26 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@paladox, your interactions with this upstream have consistently wasted our time and energy.

I warned you about untested patches in D14813. D15597 obviously hasn't been tested either since it misspells --quiet.

You need to read Support Resources, Contributing Bug Reports, and Providing Reproduction Steps very carefully and follow the rules they set out. You need to apply common sense, critical thinking, and basic software engineering skills in interactions with this upstream. If you are unable to do these things, you need to stop interacting with the upstream.

Apr 3 2016, 8:19 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

I tried to reproduce everything. I have never personally developed on Windows, so everything was done from scratch.

Apr 3 2016, 7:45 PM · Windows, Bug Report (Needs Information), Arcanist
epriestley added a comment to T10717: arc land does not work on windows.

@paladox, your interactions with this upstream have consistently wasted our time and energy.

Apr 3 2016, 7:30 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad it seems to be working now

Apr 3 2016, 7:20 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

And doing

Apr 3 2016, 6:23 PM · Windows, Bug Report (Needs Information), Arcanist
avivey added a project to T10717: arc land does not work on windows: Windows.
Apr 3 2016, 6:23 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

Oh ok, I'm not sure why I'm getting it and your not.

Apr 3 2016, 6:16 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

We still need to be able to reproduce the bug locally.

Apr 3 2016, 6:06 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad or we could use exec for windows and proc_open for everyone else.

Apr 3 2016, 6:01 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad yes. But php wouldn't stop working like that because of the windows version unless it would.

Apr 3 2016, 5:54 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Aren't you running a beta of Windows?

Apr 3 2016, 5:53 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

Could we add something to bypass proc_open since it seems to be a bug on windows.

Apr 3 2016, 5:49 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

Yep it is passthru proc_open(): proc_open(): CreateProcess causing the problem.

Apr 3 2016, 5:47 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad ive tested it now on your test differential still same problem, I'm using revision https://github.com/phacility/arcanist/commit/57f6fb59d73994d90cd94143787424ce0fdbf73b so it would have failed with proc_open if I was to use the latest revision.

Apr 3 2016, 5:41 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad I will test now.

Apr 3 2016, 5:37 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Can you reproduce against our test repositories here? See rGITTEST

Apr 3 2016, 5:37 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

It seems arc land carn't find any revisions I created or wont create any.

Apr 3 2016, 5:36 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad how do I get the Arcanist version.

Apr 3 2016, 5:36 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Windows / Git / PHP / Arcanist versions would be a good start.

Apr 3 2016, 5:32 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad what details do I give you about my envrioment, because it is setup the same as everyone else.

Apr 3 2016, 5:22 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

when I run arc land it doesn't detect the revision so I have to manually select it.

Apr 3 2016, 5:22 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Understood, but we still need the details to be able to see and fix the bug locally. Without knowing how or why your environment is different than ours, we have no means of identifying and verifying a fix.

Apr 3 2016, 5:18 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad the installer just installs the files it doesn't do anything else so all the files are from phabricator except from installer.

Apr 3 2016, 5:16 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

This is just way too far off the map as far was what we take under "Free Support" and "Bug Report". Specifically:

Apr 3 2016, 5:07 PM · Windows, Bug Report (Needs Information), Arcanist
chad closed T10717: arc land does not work on windows as Invalid.

Use the windows installer https://github.com/paladox/Arcanist-installer-for-windows

Apr 3 2016, 4:57 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad ok.

Apr 3 2016, 4:52 PM · Windows, Bug Report (Needs Information), Arcanist
chad reopened T10717: arc land does not work on windows as "Open".

OK, I'll leave it open and wait for reproduction steps. Right now the steps provided aren't reproducing in my local environment. Please take the time to include full steps to reproduce, as if you were setting up a whole new computer to isolate the cause. This is what we'll be doing. Loading a VM with your same version, installing the same PHP scripts, etc, etc.

Apr 3 2016, 4:47 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.
In T10717#168249, @chad wrote:

Is this what you installed? From the page it looks clear that 7.0.4 is a Developer install, not a Stable/Production Install.

pasted_file (295×984 px, 47 KB)

Apr 3 2016, 4:31 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad I get it even on php 5.6

Apr 3 2016, 4:30 PM · Windows, Bug Report (Needs Information), Arcanist
chad merged T10717: arc land does not work on windows into T9640: Make Phabricator compatible with PHP7.
Apr 3 2016, 4:29 PM · Prioritized, Restricted Project, Infrastructure
chad merged task T10717: arc land does not work on windows into T9640: Make Phabricator compatible with PHP7.
Apr 3 2016, 4:29 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

We don't support PHP7, see my comments above for related tasks. I'll add this as another example though. Thanks for verifying!

Apr 3 2016, 4:28 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

Is this what you installed? From the page it looks clear that 7.0.4 is a Developer install, not a Stable/Production Install.

Apr 3 2016, 4:26 PM · Windows, Bug Report (Needs Information), Arcanist
• paladox added a comment to T10717: arc land does not work on windows.

@chad ok ive installed php 7 64bit from http://windows.php.net/download/ and this is the error I still get

Apr 3 2016, 4:26 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

More specifically, we have no other reports on Windows issues with arcanist when installed in a supported environment.

Apr 3 2016, 4:19 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

@chad the php is based on the official one. Just modified a bit to run with wamstack. Unless it is not modified.

Apr 3 2016, 4:17 PM · Windows, Bug Report (Needs Information), Arcanist
epriestley added a revision to T7303: Provide OAuth access to Conduit: Unknown Object (Differential Revision).
Apr 3 2016, 4:13 PM · Restricted Project, Conduit, Restricted Project
epriestley added a revision to T7673: Make "Log Out" on Phacility instances behave in a way better aligned with user expectation: Unknown Object (Differential Revision).
Apr 3 2016, 4:13 PM · Auth, Phacility
• paladox added a comment to T10717: arc land does not work on windows.

@chad the php is based on the official one. Just modified a bit to run with wamstack. Unless it is not modified.

Apr 3 2016, 4:03 PM · Windows, Bug Report (Needs Information), Arcanist
chad added a comment to T10717: arc land does not work on windows.

We don't have PHP7 verified/supported (see T9888, T9640) and we don't support third party installers. In order to accept this you'll need to:

Apr 3 2016, 3:58 PM · Windows, Bug Report (Needs Information), Arcanist
lpriestley added a subtask for T6526: Implement "Badges" or "Medals", for annotating special users: T10702: "Award Badge" action on user profiles should use a typeahead for badge selection.
Apr 3 2016, 3:57 PM · Restricted Project, Badges (v0), People
lpriestley added a parent task for T10702: "Award Badge" action on user profiles should use a typeahead for badge selection: T6526: Implement "Badges" or "Medals", for annotating special users.
Apr 3 2016, 3:57 PM · Badges (v0)
lpriestley added subtasks for T6526: Implement "Badges" or "Medals", for annotating special users: T10671: Implement `badges.search` and `badges.edit` Conduit API methods, T10690: Allow badges to be searched by name.
Apr 3 2016, 3:55 PM · Restricted Project, Badges (v0), People
lpriestley added a parent task for T10671: Implement `badges.search` and `badges.edit` Conduit API methods: T6526: Implement "Badges" or "Medals", for annotating special users.
Apr 3 2016, 3:55 PM · Restricted Project, Conduit, Badges (v0)