package com.perforce.p4simulink; /** * Copyright (C) 2014 Perforce Software. All rights reserved. * * Please see LICENSE.txt in top-level folder of this distribution. */ import java.awt.Frame; import java.io.File; import java.io.IOException; import java.util.ArrayList; import java.util.Collection; import java.util.Collections; import java.util.EnumSet; import java.util.HashSet; import java.util.List; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; import com.mathworks.cmlink.api.ApplicationInteractor; import com.mathworks.cmlink.api.ConfigurationManagementException; import com.mathworks.cmlink.api.InteractorSupportedFeature; import com.mathworks.cmlink.api.Terminator; import com.mathworks.cmlink.api.customization.CustomizationWidgetFactory; import com.mathworks.cmlink.api.version.r14a.CMInteractor; import com.perforce.p4java.client.IClient; import com.perforce.p4java.core.ChangelistStatus; import com.perforce.p4java.core.IChangelist; import com.perforce.p4java.core.file.FileSpecBuilder; import com.perforce.p4java.core.file.IFileSpec; import com.perforce.p4java.server.ServerStatus; import com.perforce.p4simulink.connection.P4Identifier; import com.perforce.p4simulink.connection.P4Server; import com.perforce.p4simulink.connection.P4Uri; import com.perforce.p4simulink.connection.P4UriType; public class P4Interactor extends P4Server implements CMInteractor { protected static final Logger log = LogManager.getLogger(P4Interactor.class .getName()); protected final Frame parentFrame; protected IClient client; private int changeID; public P4Interactor(ApplicationInteractor applicationInteractor, P4Uri uri) throws P4CMException { log.debug("Construct P4Interactor() - {}", uri.getUri(P4UriType.P4)); setP4Uri(uri); parentFrame = applicationInteractor.getParentFrame(); terminator = applicationInteractor.getTerminator(); } @Override public void connect() throws ConfigurationManagementException { log.debug("connect()"); // connect to server try { openConnection(); } catch (Exception e) { return; } // login to server login(parentFrame); // Fetch client client = fetchClient(); } @Override public void disconnect() throws ConfigurationManagementException { try { if (isReady()) { server.disconnect(); } } catch (Exception e) { log.error(e); throw new ConfigurationManagementException(e); } } @Override public String getShortSystemName() { return "Perforce"; } @Override public String getSystemName() { P4Identifier identifier = new P4Identifier(); String product = identifier.getProduct(); String version = identifier.getVersion(); return product + " (" + version + ")"; } @Override public boolean isFeatureSupported( InteractorSupportedFeature interactorSupportedFeature) { return supportedFeatures.contains(interactorSupportedFeature); } @Override public boolean isReady() { return server != null && server.isConnected() && server.getStatus() == ServerStatus.READY; } protected void setChangeID(int change) { changeID = change; } protected int getChangeID() { // exit early if default change is selected if (changeID == 0) { return changeID; } // check change can be used try { IChangelist change = server.getChangelist(changeID); // check if pending ChangelistStatus status = change.getStatus(); if (status != ChangelistStatus.PENDING) { changeID = 0; return changeID; } // check ownership String owner = change.getUsername(); if (!getUser().equalsIgnoreCase(owner)) { changeID = 0; return changeID; } // check client workspace String currentID = client.getName(); String clientID = change.getClientId(); if (!currentID.equals(clientID)) { changeID = 0; return changeID; } } catch (Exception e) { changeID = 0; return changeID; } return changeID; } // ################################################################################ // list of supported features private static final Collection<InteractorSupportedFeature> supportedFeatures = EnumSet .of(InteractorSupportedFeature.CONNECTION); protected Terminator terminator; protected ApplicationInteractor applicationInteractor; public void setTerminator(Terminator terminator) { this.terminator = terminator; } @Override public void buildCustomActions(CustomizationWidgetFactory widgetFactory) { log.debug("buildCustomActions()"); } /** * Converts a collection of files into a list of IFileSpec objects. Any * directories contained in the input list are expanded into a list of files * contained within them (if any). * * @param files * Collection of files for which we want to get file specs. Note * that directories are expanded into the list of files contain * in them (if any). * @return List of IFileSpec objects for the input list. */ public List<IFileSpec> toSpec(Collection<File> files) { List<IFileSpec> specs = new ArrayList<>(); try { HashSet<String> filePaths = new HashSet<>(); for (File file : files) { // adding a directory means we should add the files under it if (file.isDirectory()) { filePaths.add(file.getCanonicalPath() + File.separator + "..."); } else { String filename = file.getCanonicalPath(); if (filename.contains("#") || filename.contains("@") || filename.contains("%")) { filename = filename.replace("%", "%25"); filename = filename.replace("@", "%40").replace("#", "%23"); } // regular file filePaths.add(filename); } } if (!filePaths.isEmpty()) { specs = FileSpecBuilder.makeFileSpecList(filePaths .toArray(new String[filePaths.size()])); } } catch (IOException e) { log.error("IO Exception building list of file specs: " + e.getLocalizedMessage()); } return specs; } public List<IFileSpec> toSpec(File file) { return toSpec(Collections.singletonList(file)); } }
# | Change | User | Description | Committed | |
---|---|---|---|---|---|
#3 | 11925 | Paul Allen | Select pending changelist from table. | ||
#2 | 11899 | Paul Allen | Inprogress development - moved work to branch | ||
#1 | 11898 | Paul Allen |
Populate -o //guest/perforce_software/p4sl/main/... //guest/paul_allen/p4sl/.... |
||
//guest/perforce_software/p4sl/main/src/main/java/com/perforce/p4simulink/P4Interactor.java | |||||
#28 | 11681 | Paul Allen |
Fixes to unit tests. - Fixed paths for testMove in OverrideTests - Removed adding of target before a move. - Closing of resources; writer - Added missing serialisation ID - removed unused variables. |
||
#27 | 11672 | Paul Allen |
Tidy up formatting and imports. No functional change. |
||
#26 | 11664 | Paul Allen |
New build for Mathworks to look at. Transferred from p4://perforce.perforce.com:1666@991739 |
||
#25 | 11663 | Paul Allen |
Fixing issues reported by Tim Hosey at Mathworks: fixed check project button by making getFileState function use the client root instead of a list of files for large file sets. Also investigated error messaging formats and determined that they and the "could not create new file" error are both controlled by MATLAB and not P4Simulink. All calls to "p4 reconcile" will strip the "No file(s) to reconcile." message as well. Transferred from p4://perforce.perforce.com:1666@991730 |
||
#24 | 11662 | Paul Allen |
Fix NPE in changelist selector, remove reference to "Simulink project" from error messages. Transferred from p4://perforce.perforce.com:1666@988672 |
||
#23 | 11661 | Paul Allen |
Finishing up unit tests. Transferred from p4://perforce.perforce.com:1666@985194 |
||
#22 | 11659 | Paul Allen |
Performance improvements, continued: improved performance of getFileState() by replacing the client haveList call with a direct server command. Also improved isLatest by removing fstat calls and replacing with simpler calls to diff and sync. Transferred from p4://perforce.perforce.com:1666@983506 |
||
#21 | 11656 | Paul Allen |
Additional functionality from Scania meeting, including current changelist selector (still needs descriptions added) as well as .p4ignore editor. Transferred from p4://perforce.perforce.com:1666@978388 |
||
#20 | 11655 | Paul Allen |
Updating for Scania demo. Transferred from p4://perforce.perforce.com:1666@977338 |
||
#19 | 11653 | Paul Allen |
Fixes for R2014b of MATLAB/Simulink, including: * No longer prompting for connection information in isDirSandboxForThisAdapter * Fixed up commit logic, so it no longer uses the current changelist * Added back logic to walk the directory tree to find a .p4config file, rather than needing one in the sandbox directory * Added new buttons for re-connecting to p4d, and editing .p4config * Investigated conflict issues - fixed some, better logic coming * Removed .p4ignore generation - wrong spot for it Transferred from p4://perforce.perforce.com:1666@976888 |
||
#18 | 11652 | Paul Allen |
Added boilerplate to code, fixed build issues (NPEs on plugin load), removed Mathworks classes from plugin jar, fixed delete vs missing issue on remove() call. Transferred from p4://perforce.perforce.com:1666@970498 |
||
#17 | 11651 | Paul Allen |
Finalizing alpha code changes - adding jar for release. Transferred from p4://perforce.perforce.com:1666@962486 |
||
#16 | 11650 | Paul Allen |
Adding README.txt, added revision number changes for files that have changed in the depot (have/head style) and added build version and type to build process as well as version string within the plugin. Transferred from p4://perforce.perforce.com:1666@959461 |
||
#15 | 11648 | Paul Allen |
Final changes after demo with PM, including removal of depot path from the connection dialogue, and fixing of the p4 print issue. Transferred from p4://perforce.perforce.com:1666@958518 |
||
#14 | 11647 | Paul Allen |
New project creation fixes. Transferred from p4://perforce.perforce.com:1666@958305 |
||
#13 | 11645 | Paul Allen |
Refactoring and additional work on UI: Removed Connection class and went with global server/client objects. Implemented panes for connection, depot path and passwords. Passwords are only requested when required. Transferred from p4://perforce.perforce.com:1666@957149 |
||
#12 | 11644 | Paul Allen |
Paul's changes: Refactor getAddress and getAddressForConfig to getPort and getP4JavaURI. There were a few cases when the wrong value was written + getPort was not working. Password had a null password object (now initilised) P4PaswordPane needs some work - must talk to Perfoce to see if it needs a pass or is already logged in. There are still sync -f (this is very bad) Think I miss understood the checkout method after finding and reading the JavaDoc. You were right to start. Looks like a p4 edit. Would like to set a break point and find out when it is used. Elliot's changes: * Changed export() to use p4 'print' * Connect/login now looks for P4TICKETS env var * Config no longer stores P4PASSWD or P4TICKETS * Removed P4TICKET functionality (was only for P4Jenkins) * Ensured we're using a global IServer object * checkout() now uses editFiles() (opens all files for edit) * Changed all "sync -f" to plain "sync" * Removed final client update with force flag * Removed error dialogues regarding login issues * Investigated use of P4IGNORE - I'll need to write my own filter, no big deal Transferred from p4://perforce.perforce.com:1666@955247 |
||
#11 | 11643 | Paul Allen |
Unshelved from pending changelist '953729': Paul: Added: - getWorkspaceState() -- run a p4 status - getOpenedState() -- run p4 opened - getIgnoreState() -- run p4 add -n Modified: - add() -- seemed to call reopen? - P4FileState -- simple constructor for state Elliot: * global client and changelist object within the adapter * removed repeated client updates and changelist refreshes * removed 'reopen' in add * added calls to 'edit' where appropriate file states are found * made 'checkout' a pure sync, with no edits * commented a bunch of the @Override methods * added a call to haveList in getFileState so we can see versions of files in the project * changed the connection pane to look like p4v's * refactored a bit of the dialogue code, added calls to prompt for password (like p4v) * changed all log.trace() calls to log.debug() so the log file messiness can be controlled more easily Functionality that now works properly: * open project from source control, with properly-configured .p4config file * add new project to source control, with properly-configured .p4config file * add * edit * commit * revert * compare reversions (opens Simulink diff tool) Things to work on: * dialogues (particularily the password one, but the connection one is a bit dodge-y) * remove from project (possibly always making the Simulink project meta-data editable will fix this - it *almost* works) - the current issue is Simulink needs to edit the meta-data, but Perforce has made the files read-only, so Simulink throws up on itself Transferred from p4://perforce.perforce.com:1666@954622 |
||
#10 | 11642 | Paul Allen |
More fixes from discussion with Paul and after looking at other integrations work. Transferred from p4://perforce.perforce.com:1666@953553 |
||
#9 | 11641 | Paul Allen |
Fixes from chat with Paul. Transferred from p4://perforce.perforce.com:1666@953389 |
||
#8 | 11640 | Paul Allen |
Fixes to project integrity checks within Simulink. Transferred from p4://perforce.perforce.com:1666@952945 |
||
#7 | 11639 | Paul Allen |
Integrate code from the P4Jenkins plugin, add ability to authenticate with tickets or ticket file. Worked through some issues with getting current file state. Transferred from p4://perforce.perforce.com:1666@952289 |
||
#6 | 11638 | Paul Allen |
General cleanup, fixed some logging issues, fixed getFileState. Transferred from p4://perforce.perforce.com:1666@950492 |
||
#5 | 11637 | Paul Allen |
Fix up isLatest, uncheckout, add back code that searched for a .p4rc file within the sandbox directory first, then looks in ~/.p4simulink/ and finally ~/. Transferred from p4://perforce.perforce.com:1666@950121 |
||
#4 | 11636 | Paul Allen |
Updates to integrate with Simulink, merging into one directory. Added auto submits to add, remove, etc since this is what Simulink expects. Transferred from p4://perforce.perforce.com:1666@947024 |
||
#3 | 11635 | Paul Allen |
Updated gradle build to create a "fat jar", fixed some issues with the interfaces when it runs in Simulink itself. Added connection.configuration pane, switched the adapter to use a current changelist which gives a more svn-ish feel (which the Adapter is expecting). Added some more unit tests. General re-factoring. Transferred from p4://perforce.perforce.com:1666@946607 |
||
#2 | 11634 | Paul Allen |
Debugging actual adapter functionality and starting to integrate with Simulink. Transferred from p4://perforce.perforce.com:1666@938961 |
||
#1 | 11632 | Paul Allen |
Added commons-exec library, server startup/shutdown for unit tests, fixed a bunch of NPEs. Transferred from p4://perforce.perforce.com:1666@937331 |