Page MenuHomePhabricator
Feed Advanced Search

Oct 26 2020

altendky added a comment to T5508: Add branch name parameter to Harbormaster "Make HTTP Request" build step.

I think I've got a small tidbit to add on a specific use-case with Jenkins. I had been using pipeline builds which are working nicely but I just tried to setup a multi-branch pipeline build and this requires the branch name in the Jenkins URL of the HTTP request triggering the build. I suspect in this case the proper thing to do would be to trigger and build twice. I guess that's more of an on-branch-change sort of situation than on-commit and using those triggers in Herald (they look to be present, I haven't used them) and having that information passed on to Harbormaster as mentioned could fill the need.

Oct 26 2020, 9:40 PM · Restricted Project, Harbormaster

Oct 13 2016

altendky created T11757: Hash search fails in advanced search (works at top of page).
Oct 13 2016, 8:04 PM · Search, Bug Report

Feb 28 2016

altendky added a comment to T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!.

Thanks for the discussion and for improving the clarity of my request.

Feb 28 2016, 9:11 PM · Quicksand, Feature Request, Workboards
altendky added a comment to T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!.

I do agree that commenting is usually done after reviewing the existing history. The concern expressed to me was that the extra navigational steps to get back to the workboard after commenting on a ticket were less pleasing than simply having a popover close. The only real reference I've got for this is that Trello allows viewing of the history and commenting within a popover... not that that is any justification unto itself.

Feb 28 2016, 8:34 PM · Quicksand, Feature Request, Workboards
altendky created T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!.
Feb 28 2016, 5:42 PM · Quicksand, Feature Request, Workboards

Jan 5 2016

altendky added a comment to T9441: Cygwin PHP segfaults with arcanist.

Back at work... I have the same php, php-curl, and php-json versions in Cygwin and I am still core dumping (with my questionable-or-worse distribution system I'm not sure what arcanist and libphutil versions). I then updated to arcanist b3e68c9 and libphutil adb8a9c and arc doesn't seg-fault anymore. So, it seems to me that the initial trigger has likely been removed from arcanist but the issue that made it a seg-fault instead of a nice stack trace I suppose is still there in the PHP packages.

Jan 5 2016, 1:18 PM · Windows, Arcanist

Dec 24 2015

altendky added a comment to T9441: Cygwin PHP segfaults with arcanist.

I have had this issue at work for awhile but I just happened to install fresh Cygwin at home and arcanist is working. I installed:

Dec 24 2015, 12:53 AM · Windows, Arcanist

Dec 17 2015

altendky added a comment to T4617: Base revision mismatch triggered by changes in SVN externals.

Yes, it is possible and that is what I am doing. :] You can point to an entire different server if you want, but we happen to have ^/../other_repo since the externals are on the same server.

Dec 17 2015, 1:21 PM · Subversion, Arcanist
altendky added a comment to T4617: Base revision mismatch triggered by changes in SVN externals.

Were you using externals? This is specifically about externals which can not be expected to have the same base revision since they are an entirely different repository.

Dec 17 2015, 10:43 AM · Subversion, Arcanist

Dec 16 2015

altendky added a comment to T4617: Base revision mismatch triggered by changes in SVN externals.

Sorry @aik099, I guess I totally missed your question last year. In the example, the main repository is correctly listed at revision 203 and the external repository is listed at revision 154. They are separate repositories so it would only be by chance if they ever should be the same number. I will also mention that engine is actually an external of canopen which is an external of the main repository so my propget command isn't particularly relevant.

Dec 16 2015, 8:02 PM · Subversion, Arcanist

Nov 10 2015

altendky added Q213: phriction: Slug? (Answer 244).
Nov 10 2015, 2:19 AM

Oct 26 2015

altendky added a comment to T4207: Implement (tree) subtasks, as distinct from (graph) task dependencies.

In case anyone might find it useful I put up P1828. It's my first libphutil script so it's a bit messy and you have to hardcode some of your specific server and user info but so it goes. It generates remarkup-ready dot code with a graph of the specified project's ticket dependencies.

How can we use that?
Where to put that file and how to call it within web interface of Phab.

Oct 26 2015, 2:19 PM · Maniphest

Jul 21 2015

altendky added a comment to T4207: Implement (tree) subtasks, as distinct from (graph) task dependencies.

In case anyone might find it useful I put up P1828. It's my first libphutil script so it's a bit messy and you have to hardcode some of your specific server and user info but so it goes. It generates remarkup-ready dot code with a graph of the specified project's ticket dependencies.

Jul 21 2015, 12:06 PM · Maniphest

Jul 17 2015

altendky edited P1828 Generate dot graph of ticket dependencies for a project.
Jul 17 2015, 8:23 PM

Dec 15 2014

altendky added a comment to D10977: Show SVN property changes on commit page.

Other than D8478 prefixing a '\', the latter looks the same... aka, wrong. :]

Dec 15 2014, 7:55 PM

Dec 12 2014

altendky added a comment to D10977: Show SVN property changes on commit page.

Sorry, this is all coming back to me more slowly than it should have. For the property, the output from my SVN diff-vs-0 calls were generating that message, not Phabricator.

Dec 12 2014, 1:31 PM
altendky added a comment to D10977: Show SVN property changes on commit page.

I haven't run this myself but the only regression you have mentioned compared with D8478 is that [[D8478#98991 | there is no No newline at end of file warning]] (all the other points are rendering issues). While I do think it is more important to have this fix accepted, it does seem technically correct to avoid losing that non-EOL incidentally. Based on your example XML response from SVN I made a sample script and the presence and lack of EOL are certainly exhibited in the output of SimpleXML. I'm not quite sure where it's getting lost here but somehow was not in D8478.

Dec 12 2014, 12:01 AM

Dec 10 2014

altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 7:29 PM
altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 6:28 PM
altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 5:47 PM
altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 4:56 PM
altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 3:17 PM
altendky added inline comments to D8478: Fetch SVN property changes for Diffusion.
Dec 10 2014, 1:21 PM

Nov 3 2014

altendky created T6460: Is 'Repositories' app really deprecated?.
Nov 3 2014, 1:59 PM · Phabricator

Oct 6 2014

altendky added a comment to T2999: When we detect a file as binary, explain why.

The first time you commit a good (UTF-8) file it will still list as binary because the old side of the diff is binary. The second good commit will result in a good new and old version that can be text diff'ed. Also, pure ASCII is not needed, UTF-8 is fine so you should be able to use whatever characters you need for your native language.

Oct 6 2014, 6:13 PM · Arcanist, Diffusion

Sep 23 2014

altendky added a comment to T6170: How should I create audits without always emailing all project members.

I agree whole-heartedly with your comments about pre-commit review. We do some of that but mostly not. I am picking my battles and already fighting on too many fronts. Even audits are optional at best.

Sep 23 2014, 6:51 PM · Audit
altendky added a comment to T6170: How should I create audits without always emailing all project members.

Thanks for the pointer. We do generally do this but would still like the catch-all for when we don't. Mostly I want to be able to go get a list of all commits which have not been accepted (or derived from an accepted revision) within some set restricted to certain paths in certain repositories (and since some time). Don't bug the entire team up front or whenever anybody comments on anything, but make it easy to get a list of things that need to be looked at.

Sep 23 2014, 6:01 PM · Audit
altendky created T6170: How should I create audits without always emailing all project members.
Sep 23 2014, 5:21 PM · Audit

Sep 22 2014

altendky added a comment to T4255: SVN property changes on directories not shown in Diffusion diff views?.

@davisoa I've been running each revision of D8478 since I put it in. Admittedly with only internal use by three people but I haven't been noticing any issues.

Sep 22 2014, 4:02 PM · Subversion, Diffusion

Aug 10 2014

altendky added a comment to D8478: Fetch SVN property changes for Diffusion.

Am I on the right track here? Is this of interest, or should I plan on maintaining it in a branch for myself? Thanks for your consideration.

Aug 10 2014, 12:26 AM

Aug 7 2014

altendky set the repository for D10175: Correct typo: security.alter[n]ate-file-domain to rP Phabricator.
Aug 7 2014, 1:16 PM
altendky retitled D10175: Correct typo: security.alter[n]ate-file-domain from to Correct typo: security.alter[n]ate-file-domain.
Aug 7 2014, 1:15 PM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Actually correct 'Only variables should be passed by reference' warning

Aug 7 2014, 1:03 PM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Correct 'Only variables should be passed by reference' warning

Aug 7 2014, 12:58 PM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Update for renamed file

Aug 7 2014, 12:57 PM

May 17 2014

altendky added a comment to D8478: Fetch SVN property changes for Diffusion.

I think this may be ready for another review? The security hazard should be removed and the existing diff parsing is being leveraged. Thanks for your time.

May 17 2014, 3:24 PM

Mar 25 2014

altendky added a comment to D8478: Fetch SVN property changes for Diffusion.

While trying to figure out how to get the properties with one fell swoop (SO it was suggested that SVN bindings could be better. Is there a particular reason that the SVN CLI was used instead?

Mar 25 2014, 12:49 AM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

(Actually) Get all SVN properties at once

Mar 25 2014, 12:22 AM

Mar 24 2014

altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Get all SVN properties for a revision at once

Mar 24 2014, 11:56 PM

Mar 14 2014

altendky added a project to T4617: Base revision mismatch triggered by changes in SVN externals: Arcanist.
Mar 14 2014, 7:51 PM · Subversion, Arcanist
altendky planned changes to D8478: Fetch SVN property changes for Diffusion.

I saw the other code for parsing raw diffs after I put this together so I'll go look into using it. It does seem to have some ability to deal with the svn diff output for properties (which is included). My SVN server is extra slow so I will likely benefit even more than most from reducing the number of futures. I should also have mentioned the security hazard immediately so I wouldn't forget it. Thanks for being sure to bring it back up for me.

Mar 14 2014, 3:06 AM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Remove .local copy

Mar 14 2014, 2:59 AM
altendky updated the diff for D8478: Fetch SVN property changes for Diffusion.

Correct variable names to underscore_format

Mar 14 2014, 2:44 AM

Mar 10 2014

altendky planned changes to D8478: Fetch SVN property changes for Diffusion.

This could also become a foreach (array('old', 'new') as $age) loop. It is very repetitive as it is now.

Mar 10 2014, 11:06 AM
altendky added a comment to D8478: Fetch SVN property changes for Diffusion.

After completing this (my first contribution attempt) I looked around again with a little more comfort and familiarity and it seems this whole function may be a bit backwards. It fetches the entire source for both new and old and then creates a diff from that. I added fetching of new and old properties as well. I think there is code in ArcanistDiffParser.php that may actually parse a raw SVN diff which would make this a whole lot simpler and faster... but this at least seems to work for my cases and I need to work on work stuff tomorrow. :]

Mar 10 2014, 1:47 AM
altendky edited this Maniphest Task.
Mar 10 2014, 1:34 AM · Subversion, Diffusion

Feb 19 2014

altendky added a comment to T2999: When we detect a file as binary, explain why.

Phabricator expects all text to be in UTF-8 (according to TFM), so I assumed that was the cause. I believe I used iconv but it appears recode may be better? I'm thinking I did it incorrectly and probably actually took it down to pure ASCII... :[

Feb 19 2014, 12:08 PM · Arcanist, Diffusion