Ticket #734 (closed cage: fixed)

Opened 6 years ago

Last modified 5 years ago

Branch cleanup: pdd30_install

Reported by: jkeenan Owned by: jkeenan
Priority: minor Milestone:
Component: none Version: 1.2.0
Severity: low Keywords: branch
Cc: allison Language:
Patch status: Platform: all

Description

I examined this branch to determine whether we still need to retain it at HEAD in our repository.

Determining what was done was difficult as the branch was apparently created by copying an earlier, now deleted branch: cygwin070patches. In addition, it appears that there was a lot of updating of these branches with commits to trunk.

I did an svn diff of head in this branch versus the point at which the earlier branch was created from trunk. That diff came to 3.6M, so I'm not going to attach it.

What I am attaching is the list of files which appear in that diff. 682 files were altered. It appears that the work done in this branch ranged far from the installation concerns of PDD30.

Since PDD30 has been moved from draft to PDD in the meantime (in part building on what was done in this branch), I don't see a need to retain this branch at HEAD in our repository. I propose we remove it. Any objections?

Thank you very much.
kid51

Attachments

pdd30_install.diff.changed_files.txt Download (28.1 KB) - added by jkeenan 6 years ago.
Files touched during work on pdd30_install branch

Change History

Changed 6 years ago by jkeenan

Files touched during work on pdd30_install branch

  Changed 6 years ago by jkeenan

  • status changed from new to assigned

in reply to: ↑ description ; follow-up: ↓ 3   Changed 6 years ago by doughera

Replying to jkeenan:

Determining what was done was difficult as the branch was apparently created by copying an earlier, now deleted branch: cygwin070patches. In addition, it appears that there was a lot of updating of these branches with commits to trunk.

[ . . . ]

Since PDD30 has been moved from draft to PDD in the meantime (in part building on what was done in this branch), I don't see a need to retain this branch at HEAD in our repository. I propose we remove it. Any objections?

No objections from me. There's a useful summary page at https://trac.parrot.org/parrot/wiki/Pdd30InstallTasklist.

in reply to: ↑ 2 ; follow-up: ↓ 4   Changed 6 years ago by jkeenan

  • cc allison added

Replying to doughera:

No objections from me. There's a useful summary page at https://trac.parrot.org/parrot/wiki/Pdd30InstallTasklist.

Andy: Thanks for the link to that page.

Allison: If I understand the Tasklist correctly, the only outstanding items were the two bullet points under Held for extensive platform testing. Neither of those items, AFAICT, depends on the pdd30_install branch remaining at HEAD. Do you concur? If so, then I see no further obstacle to deletion of the branch.

Thank you very much.
kid51

in reply to: ↑ 3   Changed 5 years ago by jkeenan

Replying to jkeenan:

Allison: If I understand the Tasklist correctly, the only outstanding items were the two bullet points under Held for extensive platform testing. Neither of those items, AFAICT, depends on the pdd30_install branch remaining at HEAD. Do you concur? If so, then I see no further obstacle to deletion of the branch.

Having received no further comments or objections, I removed the branch in r39717.

Thank you very much.
kid51

  Changed 5 years ago by jkeenan

  • status changed from assigned to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.